jrkerns / pylinac

An image analysis library for medical physics
https://pylinac.readthedocs.io/en/latest/
MIT License
155 stars 98 forks source link

PicketFence.plotly_analyzed_image() #514

Open YuryKirpichev opened 2 months ago

YuryKirpichev commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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