Improvements to `energyscatter.fld`
rolyp opened this issue · comments
Roly Perera commented
- Does
-Gen
(in non_renewables) correspond tooutput
(in renewables)? (If so, perhaps replacing-Gen
by-Out
would be helpful.) - Better and/or shorter names for
check_c
(since we’re omitting definition),filteredRen
,totalRensOut
,totalRensCap
(maybe omittotal
for these two) - Speed up by eliminating excessive filtering
- Absolute rather than relative indentation (maybe already dealt with on Joe’s branch)
See also:
Roly Perera commented
Importing subtasks here from explorable-viz/graphical-slicing#317
Roly Perera commented
@JosephBond From my end this would be the most useful thing for you to work on today – I’m hoping to be able to use this example but the code need some cleaning up and the performance (as already identified) is a bit slow. (The performance isn’t particularly relevant to the ICFP submission, but it still might be nice to express the code in a “natural” way that happens to be more performant!)