RMI-PACTA / PACTA_analysis

Run the PACTA analysis on EQ & CB portfolios
Other
25 stars 70 forks source link

Improve assignment of config objects to global environment #135

Open jacobvjk opened 4 years ago

jacobvjk commented 4 years ago

As described by @maurolepore in this past PR, there is a better way to load configs into the global environment than what is current PACTA_analysis practice. Once we merge the 'current_web_functionality' into the 'main' branch, this would be a good opportunity to change that across the repo

https://github.com/2DegreesInvesting/PACTA_analysis/pull/114

maurolepore commented 4 years ago

I edited the title slightly to not prescribe the implementation details. The best way should be clearer once we dive into this issue and write some tests.

maurolepore commented 4 years ago

https://github.com/2DegreesInvesting/PACTA_analysis/pull/155/commits/4aaaa4b1d59ddcac147048a5bdf1189eca9d70cf#r502613979