Closed chrisnatali closed 8 years ago
To support legacy usage of sequencer and simplify things a bit, I parameterized the demand field by which to sequence things and got rid of the EnergyMaximizeReturn model (now uses Sequencer directly).
New usage example here: https://github.com/SEL-Columbia/sequencer/blob/v0.3.0/run_sequencer.py#L82
(note that I mimic @invisibleroads method for taking args from cmd line OR json config)
I see that work is happening in branch 2.2.0, so feel free to redirect this PR there.
Thanks, Chris! For some reason, I didn't see this pull request.
To support legacy usage of sequencer and simplify things a bit, I parameterized the demand field by which to sequence things and got rid of the EnergyMaximizeReturn model (now uses Sequencer directly).
New usage example here: https://github.com/SEL-Columbia/sequencer/blob/v0.3.0/run_sequencer.py#L82
(note that I mimic @invisibleroads method for taking args from cmd line OR json config)
I see that work is happening in branch 2.2.0, so feel free to redirect this PR there.