-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rip out lots of time-adjustment functionality #507
Conversation
Old backend run stuff fails now - time to remove it all!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With regards to timeseries_data_path
→ time_data_path
I am a bit unsure if it doesn't remove too much of the meaning, but I suppose that is likely to go away anyway with #92 to be folded into the general data_sources
config
@@ -78,8 +78,8 @@ We will discuss each of these in turn, starting with ``config``: | |||
config: | |||
init: | |||
name: 'My energy model' | |||
timeseries_data_path: 'timeseries_data' | |||
subset_time: ['2005-01-01', '2005-01-05'] | |||
time_data_path: 'timeseries_data' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we'll want to remove this entirely in favour of the generic load-from-file feature
Co-authored-by: Stefan Pfenninger <[email protected]>
Co-authored-by: Stefan Pfenninger <[email protected]>
Co-authored-by: Stefan Pfenninger <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #507 +/- ##
==========================================
+ Coverage 92.72% 93.21% +0.49%
==========================================
Files 32 28 -4
Lines 4204 3817 -387
Branches 965 877 -88
==========================================
- Hits 3898 3558 -340
+ Misses 194 168 -26
+ Partials 112 91 -21
|
Partly fixes issue #356
Summary of changes in this pull request:
Reviewer checklist: