LCA-ActivityBrowser / activity-browser

GUI for brightway2
GNU Lesser General Public License v3.0
152 stars 58 forks source link

Exel database import fails unless the database name does not match #1077

Open bsteubing opened 1 year ago

bsteubing commented 1 year ago

Updating AB

What happened?

Importing Excel databases in the brightway Excel format is a powerful tool as it conserves parameters and other information.

However, in my trials with the file below (linking to ecoinvent 3.9.1, cutoff) the import fails unless I do not provide the correct name for the ecoinvent database. In that case it asks me to identify the correct database and from there it goes well. There is also an occasional crash of the AB in this, so the whole import process has some issues still. Should be a priority issue.

EV case.xlsx

Relevant errors

No response

Operating system

Windows 10

Conda environment

No response

bsteubing commented 1 year ago

To add to this... we have tried to import this Excel database now for 20+ computers and in most (if not all) cases the Excel import did not work; in some cases the AB crashed... the Excel import should undergo a solid debugging and revision in my opinion...

Zoophobus commented 1 year ago

Hi @bsteubing, apologies I don't have enough data to download the ei3.9 database at the moment. I'll download it once I've arrived and figure out the solution then.

Zoophobus commented 1 year ago

Hi @bsteubing I've been through the file and think I have found the culprit which was the unit measure in the transport, passenger car, electric process. In Linux this worked largely as expected, it gave a relinking dialog and then in the relinking method it identified the exchange as not being relinkable in an error message.

I will experiment with some of the dialog boxes in Qt for this, at present it seems to utilize more generic system wide dialogs for these exceptions, if these are unstable in Windows I'm not sure how to fix that, so customized versions might be easier.

EV_case.xlsx