cmutel
commented
7 months ago @SamiraHuber Thanks for the bug report. I am moving this issue to the correct repo.
The SimaPro CSV importer should check the lognormal values and fix them where needed. We already do this in other importers.
In the meantime, you could do:
from numbers import Number
for ds in bw2data.Database("my database"):
for exc in ds.exchanges():
if (
"uncertainty type" in exc
and exc["uncertainty type"] in (2, 3)
and (not isinstance(exc.get("scale"), Number) or exc["scale"] <= 0)
):
print(f"Fixing exchange: {exc}")
exc["previous uncertainty type"] = exc["uncertainty type"]
exc["uncertainty type"] = 0
exc["fixed nonpositive (log)normal scale"] = True
exc["previous (log)normal scale value"] = exc.get("scale")
exc["previous (log)normal loc value"] = exc.get("loc")
exc["loc"] = exc["amount"]
exc.save()
SamiraHuber
I tried to do the Monte Carlo analysis with the Agribalyse3.1 database, based on this notebook: https://github.com/brightway-lca/brightway2/blob/master/notebooks/Monte%20Carlo%20from%20Excel%20import.ipynb
The second line fails with the error: "Real, positive scale (sigma) values are required for lognormal uncertainties." The same happens in the acitivity browser. It works when using the Ecoinvent database.
I appreciate your help on how to fix it. Thanks a lot! Here are some more information:
My Agribalyse-Version: https://sustaincertcom-my.sharepoint.com/:x:/g/personal/gustave_coste_sustain-cert_com/ESwnbkR0VpRGrozFdZersVsBePolrNLmnVuCTeJbObSU4g?e=cTFO7Z&download=1
I took this from here: https://github.com/sc-gcoste/brightway-agribalyse3.1
My brightway versions: brightway2==2.4.2 bw-migrations==0.2 bw2analyzer==0.10 bw2data==1.8.1 bw2data==3.6.5 bw2io==0.8.8 bw2parameters==0.7