arfc / transition-scenarios

A repository to hold transition scenarios with Cyclus.
Other
3 stars 12 forks source link

Update to commodity_to_LWR function #133

Closed abachma2 closed 2 years ago

abachma2 commented 2 years ago

This PR includes an update to the commodity_to_LWR function of scripts/transition_metrics.py to accept a list of strings (the prototype names) instead of a single string. This is necessary to add flexibility to the function for when there are multiple prototype names that are not LWRs in the simulations. The corresponding tests have been updated, and a new test was written to specifically test this function with multiple prototype names are provided.

abachma2 commented 2 years ago

That is a good suggested change to the function docstring. I have committed that change.