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.

Sign up for GitHub

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

Jump to bottom

Add Global Sensitivity Analysis #425

Merged
merged 33 commits into from
Jul 1, 2020
Merged

Add Global Sensitivity Analysis #425

merged 33 commits into from
Jul 1, 2020

Conversation

dgdekoning
Copy link
Contributor

Adds a tab to the LCA results where the user can perform a Global Sensitivity Analysis of Monte Carlo results.

This feature will add the salib dependency.

bsteubing and others added 30 commits January 18, 2020 13:51
@bsteubing
Store A and B matrices and CFs in MonteCarloLCA for GSA
fbf608f
@bsteubing
Refactor name to MonteCarloLCA from CSMonteCarloLCA
6c9065c
@bsteubing
First version of GSA tab (just UI elements, no deeper functionality)
0f81daf
@bsteubing
include GSA code and some modifications to the montecarlo module
5a520b8
@bsteubing
GSA calculations from within AB work, next thing is now to generate s…
16093ae 
…ome output (e.g. table)
@bsteubing
Mostly functioning GSA interface
147526c
@bsteubing
Minor improvements, e.g. error catching for GSA and converting pedigr…
69b3a4b 
…ee dicts to str so they are displayed in the table
@bsteubing
removing the include pedigree uncertainties checkbox as there is curr…
4cde077 
…ently no solid way of doing this (only excluding lognormal, but that could also be defined without the pedigree).
@dgdekoning
Update recipes and test script to include salib dependency
fb9ba84
@bsteubing
Merge branch 'master' into GSA
27556b3
@bsteubing
Merge remote-tracking branch 'origin/GSA' into GSA
2630372
@bsteubing
changed "SA" to "Global SA"
25605aa
@bsteubing
Accounting for negative impacts as well
09db8a7
@bsteubing
Just to be on the safe side of calculation depth as this variable can…
ac44cf2 
…not yet be changed by the user...
@bsteubing
Merge branch 'master' into GSA
36850d6 
# Conflicts:
#	activity_browser/app/bwutils/__init__.py
#	activity_browser/app/bwutils/montecarlo.py
#	activity_browser/app/ui/tabs/LCA_results_tabs.py
#	ci/travis/recipe/meta.yaml
#	recipe/meta.yaml
@bsteubing
bug fix
c121bb6
@bsteubing
Adding better file naming, direct output of input and output GSA data…
e78c5a1 
… and catching the no data to concatenate error if there are no uncertainties as input to the GSA.
@bsteubing
Merge branch 'master' into GSA
b453783
@bsteubing
include log transformation of Y to make it work for toxicity impacts
2882a4c
@bsteubing
Merge branch 'master' into GSA
a9fdaa8 
# Conflicts:
#	activity_browser/app/ui/tabs/LCA_results_tabs.py
#	appveyor.yml
@bsteubing
Merged master into GSA branch. Found some issues with parameter uncer…
0c4a9ff 
…tainties not being propagated into A and B matrices. Moved the switch_bw_dir function to commontasks.py
@bsteubing
adding prototype of what needs to be stored for parameters in MonteCa…
1da220b 
…rloLCA for GSA
@bsteubing
adding prototype of what needs to be stored for parameters in MonteCa…
e0aeb15 
…rloLCA for GSA
@dgdekoning
Remove for-loop, separate param_exchanges
a9b1094 
by A/B matrices
@dgdekoning
Add special GSA code to parameter utils
9ebfcce
@dgdekoning
Store data correctly for processing in GSA
0182346 
And remove superfluous print statements
@dgdekoning
Get rid of the floating scenario combobox
3448e89 
As it isn't used in the GSA tab
@dgdekoning
Use matrix row/cols instead of input/output ints
6c9d321 
Also, ensure parameter exchange amounts are sorted
according to LCA matrix.
@dgdekoning
Ensure unused parameterized exchanges are dropped
fe03c38 
Improve comments
@dgdekoning
Track parameter values
7b64462 
* Methods to build and filter param dictionary

* Retrieve data from current sampling in MC param manager

* Implement these methods in the MC LCA class
@dgdekoning dgdekoning added the feature Issues/PRs related to a new feature label Jun 30, 2020
@dgdekoning dgdekoning added this to the Version 2.6.0 milestone Jun 30, 2020
@coveralls
Copy link

coveralls commented Jun 30, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-1.9%) to 54.022% when pulling 3bd5aee on GSA into 0d0fdba on master.

@dgdekoning dgdekoning mentioned this pull request Jul 1, 2020
Closed
@dgdekoning dgdekoning merged commit b0f3a18 into master Jul 1, 2020
@Zoophobus Zoophobus deleted the GSA branch December 4, 2022 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature Issues/PRs related to a new feature
Projects
None yet
Milestone
Version 2.6.0
Development

Successfully merging this pull request may close these issues.
3 participants
@dgdekoning @coveralls @bsteubing