-
Notifications
You must be signed in to change notification settings - Fork 5
Replaced LoadProfileInput
with LoadProfileTimeSeries
#1229
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
Conversation
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.
Having some questions and I'm not sure if I got the reason for these changes. Maybe we can have a talk the next days to get me on the same level, thanks :)
src/main/java/edu/ie3/datamodel/io/naming/EntityPersistenceNamingStrategy.java
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/models/timeseries/repetitive/LoadProfileEntry.java
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/models/timeseries/repetitive/RepetitiveTimeSeries.java
Outdated
Show resolved
Hide resolved
# Conflicts: # CHANGELOG.md
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.
LGTM so far, thanks for checking my questions. Could you please have a look to the sonarQube issues and then where are good to go from my side
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.
LGTM
Resolves #1228