Skip to content
New issue

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

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

Already on GitHub? # to your account

technology_input_harmonisation cannot handle technology=None #323

Closed
FlorianK13 opened this issue Aug 16, 2022 · 1 comment
Closed

technology_input_harmonisation cannot handle technology=None #323

FlorianK13 opened this issue Aug 16, 2022 · 1 comment
Labels
🐛 bug Something isn't working

Comments

@FlorianK13
Copy link
Member

technology is assumed to be a list for method="API". The default value of None causes an error.

Traceback (most recent call last):
File "C:\Users\kotthoff\Documents\Projekte\eTwin\github\open_mastr_manual_test.py", line 65, in
db.download(
File "c:\users\kotthoff\documents\projekte\etwin\github\open-mastr\open-mastr\open_mastr\mastr.py", line 186, in download
) = technology_input_harmonisation(
File "c:\users\kotthoff\documents\projekte\etwin\github\open-mastr\open-mastr\open_mastr\utils\helpers.py", line 293, in technology_input_harmonisation
if "permit" in technology:
TypeError: argument of type 'NoneType' is not iterable

@FlorianK13 FlorianK13 added the 🐛 bug Something isn't working label Aug 16, 2022
FlorianK13 added a commit that referenced this issue Aug 17, 2022
Additionally, do not use the location_to_csv anymore as it would need
a filled locations_basic table. Simply export locations_extended from bulk
download.
FlorianK13 added a commit that referenced this issue Aug 17, 2022
FlorianK13 added a commit that referenced this issue Aug 17, 2022
@FlorianK13
Copy link
Member Author

Solved in v0.11.6

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant