Feat/ecoinvent interface

will7200 commented 3 weeks ago

Integrates ecoinvent_interface package instead of eidl.
Uses a versioned biosphere database instead of the default batabase named biosphere

Closes #7

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[ ] Update the documentation, please follow the numpy style guide.
[ ] Update tests.
[ ] Categorize the PR by setting a good title and adding one of the labels: bug, feature, ui, change, documentation, breaking, ci as they show up in the changelog.
[ ] Link this PR to related issues by using closing keywords.
[ ] Add a milestone to the PR for the intended release.
[ ] Request a review from another developer.

will7200 commented 3 weeks ago

Added the bd.preferences to update on eco version page. Do we need to run UpdateBiosphere?

cmutel commented 3 weeks ago

Do we need to run UpdateBiosphere?

No. In general this isn't a great idea, and should require manual action by the user after the import complete.

will7200 commented 3 weeks ago

I ended having to integrate UpdateBiosphere for the newly versioned database created. Otherwise the end user will go through the flow of trying to import the database, only to get the update the biosphere by using 'File' -> 'Update biosphere...' message, having to go through the wizard twice to get it to work properly.

This versioned database page only gets shown to the end user if that particular database does not exist already.

cauldron / activity-browser

Feat/ecoinvent interface #16

Checklist