brightway-lca / brightway2

Metapackage for brightway2 imports and documentation
https://brightway.dev/
BSD 3-Clause "New" or "Revised" License
100 stars 37 forks source link

SimaPro export fail match with ecoinvent #27

Open MaximeAgez opened 3 years ago

MaximeAgez commented 3 years ago

Hello again,

After the bug from #26, I tried to simply remove all parameters by using the option "Convert expression to constants" when exporting with SimaPro. The import works properly:

image

but this time it's the matching with ecoinvent3.6 cut-off that seems to fail. Even though, it's supposed to be run with calling .strategies() I specifically call sp.match_database("ecoinvent3.6 cut-off", ignore_categories=True). Yet I still have unlinked exchanges:

image

and the unlinked processes are actual processes from ecoinvent3.6 cut-off, e.g :

image

Could the matching be failing because the name of the process is the "SimaPro-name" (let's call it like that), which means in the screenshot above: 'Transport, freight, lorry >32 metric ton, euro5 {RoW}| market for transport, freight, lorry >32 metric ton, EURO5 | Cut-off, U', while in ecoinvent3.6 cut-off in my AB/brightway2 the name of the process would be something like 'Transport, freight, lorry >32 metric ton, euro5'?

I'm assuming dealing with this different nomenclature is a core concept to the SimaProCSVImporter class, so I guess it must be broken somehow because of an update from SimaPro or whatnot

If needed I can provide the Excel export files.