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

NanoPlot 1.39 crashes halfway - ValueError raised from plotly #283

Closed
landmanf opened this issue Dec 2, 2021 · 6 comments
Closed

NanoPlot 1.39 crashes halfway - ValueError raised from plotly #283

landmanf opened this issue Dec 2, 2021 · 6 comments

Comments

@landmanf
Copy link

landmanf commented Dec 2, 2021

Hi Wouter,
I'm trying to get NanoPlot to work in a environment with several other dependencies and it crashes halfway before finishing all of the .html it raises an ValueError from plotly

I tried several versions of NanoPlot

Versions (1.35.5, 1.36.1,1.37.0, 1.38.0) gave " scatter() missing 1 required positional argument: 'data_frame' " which I saw in an previous issue (#268 and was fixed in 1.38.1) so I tried those newer versions (1.38.1, 1.39.0) but they gave the ValueError from plotly

Hopefully it's an easy fix :)

Added the logfile:
NanoPlot_20211202_0943.log

@wdecoster
Copy link
Owner

Hi, that seems to be a duplicate of #278, which is unfortunately unsolved as the one who reported that didn't follow up.

My first idea is that you might have an outdated plotly module, could you check the version?
python -c "import plotly ; print(plotly.version)"

@landmanf
Copy link
Author

landmanf commented Dec 2, 2021

Thank you for the quick reply.
This is the output:
<module 'plotly.version' from 'path/to/conda/envs/test_nanoplot2/lib/python3.8/site-packages/plotly/version.py'>
output to your previous command in #278 gave me a version number which is 4.1.0
I noticed now, that the underscores got formatted :)

@wdecoster
Copy link
Owner

Glad you caught my error in copy-pasting the command :-)

Could you just try upgrading plotly? At least that gives me a starting point on how to reproduce this...

@landmanf
Copy link
Author

landmanf commented Dec 2, 2021

Upgraded plotly to 5.4.0 and now NanoPlot 1.38.1 works perfect without errors. In the process of upgrading through pip it revealed the program holding plotly back which is PycoQC and says it requires plotly 4.1.0.

@wdecoster
Copy link
Owner

Aha, that is remarkable but very good to know. I will change the minimal required version for NanoPlot then to 5.4.0... Thanks for the quick feedback!

wdecoster added a commit that referenced this issue Dec 2, 2021
should solve error reported in #283 and #278
@landmanf
Copy link
Author

landmanf commented Dec 2, 2021

Interestingly/luckily enough PycoQC (v2.5.2) still works in this environment as well when only supplying the --summary_file and --output.
Thank you for your help :)!

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