Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated scenarios #1025

Merged
merged 32 commits into from
Aug 30, 2023
Merged

Conversation

Zoophobus
Copy link
Contributor

Adds features for sanity checks on scenario difference file imports. Provides multiple new testing features to help in resolving the following issues:
#861 #839 #820 #769 #746 #744 #736 #592 #562 #542 #481

Checklist

  • Update the documentation.
  • 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.

…the influence of external settings on the appearance of the AB
…he location at which these settings are now applied (at the base application level)
…es in the scenario difference file, with a popup to provide the ability to relink to a database currently loaded within the AB. Error messages are thrown if an activity cannot be relinked, usable for both technosphere and biopshere activities/flows. The search routine is labelled for updating on refactoring for bw2.5.
Improved structure of loading, fixes use of Exceptions for scenario parameter imports.
…on importing a scenario file. Addition of a save functionality for the combined scenario dataframes within the AB.
…s in the scenario files. Improves the messaging and text provided in the popup and warning messages. Enables customisation of default buttons in ABPopups (for GUI error messaging)

Added comments and documentation to the recent updated code for checking and processing scenario files.
…ario files. Includes: utilization of the AB-metadata store, progress indication and list comprehension based iteration.
…function and the ABPopup save_dataframe method: 1) Iterates only through those exchanges containing an unidentifiable database. 2) Corrects the dataseries value assignment in the internal function. 3) The separator for csv files is specified to ';'.
…ases function. 1) Passes the dataframe index to the internal function, allowing the user to save only those exchanges that cannot be linked. 2) Adds a new error to allow the AB to catch the issue and return the cursor to normal.
…and use of bad keys for accessing activities from the BW SQL databases. Part of this work involves the separation of providing the keys and running checks on the keys. If a key is not present and the metadata is correct an error is thrown, this is only dependent on the absence of a single key. The checks have also been moved to execute within the context of the sync_superstructure method of the ScenarioImportWidget class.
…ncluding throwing a more specific error that should be handled softly.
…es for importing scenario difference files. Standardises systems to resolve issues found with input files.
…ions in pandas documentation. Also restores the cursor for use in the appropriate popup windows.
…han QMessageBox). Additional icons are provided.
…tention of the loading cursor symbol. Additional fix for the use of single scenario files with self referential flows.
…es function contained a redundant reference to the relinkiing object
…r unique scenario names, when combining scenarios across files using a combinatoric approach. Adds popup windows for Errors and warnings related to the scenario names for the 'Extension' approach, for combining scenario files. Changes the location of checks for scenario exchange amounts to the superstructure.manager module and removes the superstructure.exchanges module. Provides some minor editing to messages for popup windows. Combines the updates to calculation procedures to keep aligned with recent merges.
…olumnsWarning'. Contains significant changes to the manager module: Changes the execution of checking the scenario exchange values. Changes the use of scenario expansion errors. Vectorizes the approach used in merge_flows_to_self. Also contains changes to the updated routines for updating scenario calculations.
@Zoophobus Zoophobus added bug Issues/PRs related to bugs feature Issues/PRs related to a new feature info:UI labels Aug 30, 2023
@coveralls
Copy link

Coverage Status

coverage: 50.589% (-1.1%) from 51.683% when pulling d7c7861 on Zoophobus:updated-scenarios into 4488bb8 on LCA-ActivityBrowser:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues/PRs related to bugs feature Issues/PRs related to a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants