Skip to content
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

Document addition of time series data with transformed time series. #935

Closed
ktehranchi opened this issue Dec 7, 2022 · 2 comments · Fixed by #1148
Closed

Document addition of time series data with transformed time series. #935

ktehranchi opened this issue Dec 7, 2022 · 2 comments · Fixed by #1148
Assignees

Comments

@ktehranchi
Copy link

I’m attempting to add a new HydroEnergyReservoir with a hydroCommitmentReservoirBudget formulation to the modified-RTS-GMLC-DA system from casebuilder. I intend to use the hydrobudget as a daily “MWH” budget that a new hydroEnergyReserviour can dispatch each day…. My TS CSV has two columns with 365 values of the daily energy budget.

The current error- is "InfrastructureSystems.ConflictingInputsError("forecast horizon 1 does not match system horizon 48". Jose has identified the issue is that the static time-series would need to be in the deterministic forecast format to be added to the system because this case-builder system is modified to include forecasts already.

Apologies for my poor explanation. Code attached.

hydroSim_2016_@.csv
issue.txt

@jd-lara
Copy link
Member

jd-lara commented May 16, 2023

This issue is related to trying to add data to a system with transformed time series. This is mostly a documentation issue on how to handle these cases

@jd-lara jd-lara changed the title Unable to add static time-series to new component in case-builder system with modified data Document addition of time series data with transformed time series. May 16, 2023
@jd-lara
Copy link
Member

jd-lara commented Jun 24, 2024

@kdayday and @daniel-thom can we. confirm this case is covered in the documentation? If yes please close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants