-
Notifications
You must be signed in to change notification settings - Fork 52
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
UnboundLocalError #160
Comments
Also important to Report that the python env ist completely clean and that the code works in Google colab but i need it to be able to run in local |
Thanks, we should have caught this (seems to be coming from a change at the NCBI end though). Can you try pulling from the master branch and retry? pip install git+https:://github.com/saketkc/pysradb |
Works for me, thanks! |
Hi. Thanks for fixing this problem. We also encountered it just today and wanted to report this.. Not neccessary at this point anymore. But here's our solution: it seems like edit: xmltodict 0.13.0 uses dict instead of OrderedDict now (as reported in the issue tracker of xmltodict) |
Thanks. This should now be fixed in v.1.4. |
Describe the bug
`UnboundLocalError Traceback (most recent call last)
Input In [7], in <cell line: 3>()
1 from pysradb import SRAweb
2 db = SRAweb()
----> 3 df = db.sra_metadata('SRP098789')
4 df.head()
File ~\PycharmProjects\Tools\venv\lib\site-packages\pysradb\sraweb.py:516, in SRAweb.sra_metadata(self, srp, sample_attribute, detailed, expand_sample_attributes, output_read_lengths, **kwargs)
514 experiment_record["sample_title"] = exp_sample_name
515 experiment_record["instrument"] = exp_instrument
--> 516 experiment_record["instrument_model"] = exp_platform_model
517 experiment_record["instrument_model_desc"] = exp_platform_desc
518 experiment_record["total_spots"] = exp_total_spots
`
To Reproduce
from pysradb import SRAweb
db = SRAweb()
df = db.sra_metadata('any data')
df.
Desktop (please complete the following information):
Additional context
I just used this oackage a week ago with this exact structure and worked fine
The text was updated successfully, but these errors were encountered: