-
Notifications
You must be signed in to change notification settings - Fork 326
0.36.0 release not compatible with sphinx autoapi plugin #4871
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
Comments
Are you using the QCoDeS sphinx plugin ? The only other thing that I can think of is the lazy loading that was added with #4788 |
Hi @jenshnielsen, could you share a link to the plugin you mentioned? I'm not able to find it myself, unfortunately. I will investigate the lazy loading thing. |
importing h5py directly solves the issue 😄. For completeness, I added the line |
Great to hear that we found this issue. It would probably be good to report it over here https://github.com/scientific-python/lazy_loader/issues if we can find a way to reproduce it a bit simpler. We are using this https://github.com/QCoDeS/Qcodes/blob/master/qcodes/sphinx_extensions/parse_parameter_attr.py extension to be able to document Parameters on instruments a bit better (instance attributes defined in |
Thanks :) created an issue over here scientific-python/lazy-loader#37 |
We use sphinx to create documentation for our
quantify-scheduler
andquantify-core
packages. It looks like when we haveqcodes==0.36.0
installed the building of the documentation fails.Unfortunately, the stack trace is not informative in where and why it happens:
just that it occurs right after autoapi is running.
After some trial and error I found out that if I do a
pip install --no-deps qcodes<0.36.0
the build succeeds, and when I do apip install --no-deps qcodes==0.36.0
it fails.Unfortunately, I don't have a minimal working example other than pip installing our quantify package and building the documentation locally:
this will fail the build with the error I pasted above. After doing a
pip install --no-deps qcodes<0.36.0
it succeeds.The text was updated successfully, but these errors were encountered: