Fix naming of some activity/capacity-related things#441
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #441 +/- ##
=======================================
Coverage 95.11% 95.11%
=======================================
Files 32 32
Lines 4751 4752 +1
Branches 4751 4752 +1
=======================================
+ Hits 4519 4520 +1
Misses 119 119
Partials 113 113 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I've realised we still use the term |
tsmbland
left a comment
There was a problem hiding this comment.
A couple of comments, otherwise looks good
| @@ -1,4 +1,4 @@ | |||
| process_id,start_year,end_year,capital_cost,fixed_operating_cost,variable_operating_cost,lifetime,discount_rate,cap2act | |||
| process_id,start_year,end_year,capital_cost,fixed_operating_cost,variable_operating_cost,lifetime,discount_rate,capacity_to_activity | |||
There was a problem hiding this comment.
I think cap2act is used in other energy systems models so we might consider keeping this. Maybe @ahawkes can comment?
There was a problem hiding this comment.
Oh, interesting. I assumed it was a MUSE-ism. If spelling it out sounds silly to economists, then let's leave it as is.
We could always spell it out in the code and leave it abbreviated in the CSV column names if we wanted to.
There was a problem hiding this comment.
@ahawkes do you have an opinion on this? It'd be good to get this PR merged in some form soonish so that we don't get merge conflicts.
|
|
||
| /// Maximum activity for this asset in a year. | ||
| /// | ||
| /// This was referred to as `capacity_a` in MUSE 1.0. |
There was a problem hiding this comment.
Nope, no such thing as capacity_a in muse1
There was a problem hiding this comment.
Oh ok. Is it just referred to as maximum activity or something?
Description
I've a renamed a couple of activity-related things for consistency/clarity:
cap2actfield and CSV column tocapacity_to_activity. We've previously expanded out these kinds of abbreviations elsewhere and I think it makes things more readable. (I suppose we could name it something else, likeactivity_per_capacityinstead)Type of change
Key checklist
$ cargo test$ cargo docFurther checks