brightway-lca / brightway2-io

Importing and exporting for the Brightway LCA framework
BSD 3-Clause "New" or "Revised" License
26 stars 40 forks source link

ValueError when executing bw2setup() #277

Open mfastudillo opened 4 hours ago

mfastudillo commented 4 hours ago

I am testing bw2io 0.9.DEV38 and if I try to run bw2setup in a new project I get a ValueError

Can't understand elementary flow identifier ['biosphere3', '9990b51b-7023-4700-bca0-1a32ef921f74']

I am using bw2data version (4, 0, 'dev55')

@tngTUDOR

mfastudillo commented 3 hours ago

the error is raised here

I am not sure why the code seems to expect a tuple... if I read the source of the methods the pair (database,code) is a list, not a tuple :thinking:


lcia_path = (Path.cwd()/'bw2io'/'data'/'lcia'/"lcia_39_ecoinvent.zip")

with zipfile.ZipFile(lcia_path, mode="r") as archive:
            data = json.load(archive.open("data.json"))

for method in data:
    method["name"] = tuple(method["name"])

for d in data:
    for e in d['exchanges']:

        assert isinstance(e['input'],list)
cmutel commented 3 hours ago

bw2setup is deprecated: https://docs.brightway.dev/en/latest/content/cheatsheet/projects.html#creating-initial-project-data. You should either start from one of the reference projects or, if using an existing project, use ecoinvent_interface to install the biosphere data and LCIA implementations you want.

mfastudillo commented 2 hours ago

ah ok, I wasn't aware of that... and if we add a warning about the method being deprecated ?