-
Notifications
You must be signed in to change notification settings - Fork 25
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
Multiple Maximum Generation and Pumping for Hydro (Merging with develop branch) #1723
Changes from 119 commits
2b03609
e25aea9
9bbe22e
ab3085f
f2bd3ac
5a80015
f602cc6
22d74c4
34a314b
0fb26de
e2e963a
a4d33db
eee6d08
a4c1da8
827cfb5
07ce46b
4b08e13
ede8efc
598a474
4554d23
94ca6b7
2ab16dc
13db210
04b6104
1820fda
94698f1
75b6263
91435e9
c672e3a
118db9a
7ee9ccf
4b165d2
6cad434
401e21a
75cb113
bcdc18b
8e35773
4abc817
c104436
0667a39
7138ced
cabdc79
94784ea
c8f2050
a33099d
21cd52e
7fa96b3
b5f0573
aadaa26
8ff8737
0d9e8e5
a4f00a2
c142ee9
644c50b
172d8a9
68d4299
1ec8f3e
552b48a
be21fda
578c6e5
fb0c474
a9e6519
b2899d5
596d9f4
54bbc79
2672405
f78e096
20a4738
b1874ec
32b56e5
c0643bb
838c04c
376f24d
1eb238c
fe47970
8fa59e9
25c4b23
426b3c9
9cd3f88
edd5c50
5354e87
867aa5b
81850af
030af42
7c5ddc5
f1c3673
d21f858
6c01298
fd1e1eb
baa6994
00b8592
cc82545
0938768
7707ac3
4b3274e
bf5c1d4
df5b115
9cdee5c
d3bf301
7988dde
3da3b13
747ca30
f0056dd
c562812
8b4a1a5
af4ad02
0697e89
4463240
2a52a46
468bd2e
8d62fa9
3eff63d
4232fcc
d6c2d13
07e112f
092abb4
b3c9cdb
fe6cbe7
a8152ed
3f4530c
0b50919
8529ed8
f7304a0
4d0e5a5
00c1ba3
e368fe4
e2b9339
ec9f4dc
b4b03bc
ab4e9b2
5e5edbf
6217c8d
292989a
a186bad
5582406
8ce0cff
3415b2e
bee6e82
5bea0ca
4392ccc
016c186
f1e84ed
da9e697
0a2b300
30ce902
2bcf620
837afbd
acdff1a
f098190
05fe9e6
c06b3b8
6fab9c1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -141,5 +141,4 @@ uint64_t TimeSeries::memoryUsage() const | |
{ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This PR contains console logs. Please review or remove them. |
||
return timeSeries.memoryUsage(); | ||
} | ||
|
||
} // namespace Antares::Data |
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -897,12 +897,41 @@ static bool AreaListLoadFromFolderSingleArea(Study& study, | |||||||
buffer.clear() << study.folderInput << SEP << "hydro" << SEP << "prepro"; | ||||||||
ret = area.hydro.prepro->loadFromFolder(study, area.id, buffer.c_str()) && ret; | ||||||||
} | ||||||||
if (area.hydro.series && (!options.loadOnlyNeeded || !area.hydro.prepro)) // Series | ||||||||
|
||||||||
auto* hydroSeries = area.hydro.series; | ||||||||
if (!options.loadOnlyNeeded || !area.hydro.prepro) // Series | ||||||||
{ | ||||||||
buffer.clear() << study.folderInput << SEP << "hydro" << SEP << "series"; | ||||||||
ret = area.hydro.series->loadFromFolder(study, area.id, buffer) && ret; | ||||||||
ret = hydroSeries->loadGenerationTS(area.id, buffer, study.header.version) && ret; | ||||||||
|
||||||||
hydroSeries->EqualizeGenerationTSsizes( | ||||||||
area, study.usedByTheSolver, study.gotFatalError); | ||||||||
} | ||||||||
|
||||||||
if (study.header.version < 870) | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Still a lot of 870 in the code base, we should take care of this and turn them to 890. |
||||||||
{ | ||||||||
buffer.clear() << study.folderInput << SEP << "hydro"; | ||||||||
|
||||||||
HydroMaxTimeSeriesReader reader; | ||||||||
ret = reader(buffer, area, study.usedByTheSolver) && ret; | ||||||||
} | ||||||||
|
||||||||
if (study.header.version >= 870) | ||||||||
{ | ||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done! |
||||||||
buffer.clear() << study.folderInput << SEP << "hydro" << SEP << "series"; | ||||||||
ret = hydroSeries->LoadMaxPower(area.id, buffer) && ret; | ||||||||
|
||||||||
if (study.usedByTheSolver) | ||||||||
{ | ||||||||
hydroSeries->EqualizeMaxPowerTSsizes(area, study.gotFatalError); | ||||||||
} | ||||||||
else | ||||||||
hydroSeries->setHydroModulability(area); | ||||||||
} | ||||||||
|
||||||||
hydroSeries->resizeTSinDeratedMode( | ||||||||
study.parameters.derated, study.header.version, study.usedByTheSolver); | ||||||||
|
||||||||
++options.progressTicks; | ||||||||
options.pushProgressLogs(); | ||||||||
} | ||||||||
|
@@ -1556,9 +1585,7 @@ void AreaList::removeLoadTimeseries() | |||||||
|
||||||||
void AreaList::removeHydroTimeseries() | ||||||||
{ | ||||||||
each([](Data::Area& area) { | ||||||||
area.hydro.series->reset(); | ||||||||
}); | ||||||||
each([](Data::Area& area) { area.hydro.series->reset(); }); | ||||||||
} | ||||||||
|
||||||||
void AreaList::removeSolarTimeseries() | ||||||||
|
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.
revert this ?
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.
Accidentally pushed. I've tried to delete folder, but that is not the solution. How to resolve this?
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 took care of this, it has now disappear
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.
Thanks!
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.
Thanks a lot!! It was killing us, how to fix it!