Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PicketFence.plotly_analyzed_image() #514

Open
YuryKirpichev opened this issue Sep 4, 2024 · 2 comments
Open

PicketFence.plotly_analyzed_image() #514

YuryKirpichev opened this issue Sep 4, 2024 · 2 comments

Comments

@YuryKirpichev
Copy link

Firstly I wanna thank you for this impressive and powerful library.
But I wanna make a bug report for pylinac==3.26.0 at PicketFence test analisation with option separate_leaves=True.

pf = PicketFence.from_demo_image()
pf.analyze(tolerance=0.5, action_tolerance=0.25, separate_leaves=True,)
pf.plotly_analyzed_image()
Error:
plotly.graph_objs._figure.Figure.add_scatter() got multiple values for keyword argument 'line_color'

while with separate_leaves=False works perfect

There is an example at
https://colab.research.google.com/drive/1YkLZdr8W2mjKbPoC8Wr7ItyZmfDogGGk?usp=sharing

But the behaviour is similar on a local machine as well.

Do you have any suggestions?

@jrkerns
Copy link
Owner

jrkerns commented Sep 4, 2024

The plotly behavior is purposely undocumented because it's still in progress. I recommend using the matplotlib variant plot_analyzed_image until the new plotly behavior is worked out. the method and keyword arguments are likely to change.

@YuryKirpichev
Copy link
Author

Thank you very much, James Kerns

I am waiting.
Anyway if you need I will be really happy to help you with testing of these new awesome functions.

My best

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants