Multiple improvements for the scenario import panel - Githubissues

LCA-ActivityBrowser / activity-browser

GUI for brightway2

GNU Lesser General Public License v3.0

134 stars 50 forks source link

Multiple improvements for the scenario import panel #1203

Closed mrvisscher closed 5 months ago

mrvisscher commented 5 months ago

Started out as a bugfix, ended up as a series of quality of life improvements for the scenarios import panel.

Changed the scenario import toolbar to look a bit cleaner and to be more intuitive
Added some statistics on the bottom so the user can perform some sanity checks on the scenario imports
Adding and removing tables is now quicker if the tables are empty
You can now actually switch the combination type after you've imported the files
added by @marc-vdm: Resolve bug where duplicate flows would crash AB + improve excel/csv scenario exports

Demo

scenario-manager

Related issues

Fixes #1177
Fixes #1084
Fixes #1086
Fixes #1143

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Update the documentation, please follow the numpy style guide.
[x] 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.
[x] Link this PR to related issues by using closing keywords.
[x] Request a review from another developer.

coveralls commented 5 months ago

coverage: 50.927% (+0.05%) from 50.879% when pulling 611f30fbc7ced3455051c8880fe89af06bc37b85 on mrvisscher:scenario-deletion-fix into 6931bad51a55d68bdbffafcfa2f5d2ae9afa139e on LCA-ActivityBrowser:master.