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

Transfer LoadProfile parsing code from SIMONA? #1106

Open
sebastian-peter opened this issue Jun 24, 2024 · 0 comments · May be fixed by #1135
Open

Transfer LoadProfile parsing code from SIMONA? #1106

sebastian-peter opened this issue Jun 24, 2024 · 0 comments · May be fixed by #1135
Assignees
Labels
io Issues relating to input/output model Related to the data model (conceptually, not to the implementation)

Comments

@sebastian-peter
Copy link
Member

LoadProfileInput, LoadProfileEntry seem to be unused in SIMONA, which has its own implementation instead. Check whether it makes sense to transfer the reading and handling of load profile parameters into PSDM, which seems to be more suited for that.

@sebastian-peter sebastian-peter added io Issues relating to input/output model Related to the data model (conceptually, not to the implementation) labels Jun 24, 2024
@staudtMarius staudtMarius self-assigned this Jun 24, 2024
@staudtMarius staudtMarius linked a pull request Aug 9, 2024 that will close this issue
staudtMarius added a commit that referenced this issue Aug 23, 2024
…ing-code-from-SIMONA

# Conflicts:
#	CHANGELOG.md
#	src/main/java/edu/ie3/datamodel/models/timeseries/repetitive/LoadProfileInput.java
staudtMarius added a commit that referenced this issue Sep 20, 2024
staudtMarius added a commit that referenced this issue Nov 8, 2024
…om-SIMONA

# Conflicts:
#	src/main/java/edu/ie3/datamodel/io/naming/DatabaseNamingStrategy.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
io Issues relating to input/output model Related to the data model (conceptually, not to the implementation)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants