Skip to content

Included option to parse a DateTimeFormatter to the processors #799

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

Closed
wants to merge 13 commits into from

Conversation

vickybung1
Copy link
Contributor

Resolves #766

@sonatype-lift
Copy link
Contributor

sonatype-lift bot commented Apr 24, 2023

🛠 Lift Auto-fix

Some of the Lift findings in this PR can be automatically fixed. You can download and apply these changes in your local project directory of your branch to review the suggestions before committing.1

# Download the patch
curl https://lift.sonatype.com/api/patch/github.com/ie3-institute/PowerSystemDataModel/799.diff -o lift-autofixes.diff

# Apply the patch with git
git apply lift-autofixes.diff

# Review the changes
git diff

Want it all in a single command? Open a terminal in your project's directory and copy and paste the following command:

curl https://lift.sonatype.com/api/patch/github.com/ie3-institute/PowerSystemDataModel/799.diff | git apply

Once you're satisfied, commit and push your changes in your project.

Footnotes

  1. You can preview the patch by opening the patch URL in the browser.

@vickybung1 vickybung1 changed the title Included option to parse a dateTimeFormatter to the processors Included option to parse a DateTimeFormatter to the processors Apr 24, 2023
@sebastian-peter sebastian-peter added the io Issues relating to input/output label Jun 6, 2023
t-ober and others added 7 commits June 19, 2023 14:28
# Conflicts:
#	CHANGELOG.md
#	src/test/groovy/edu/ie3/datamodel/io/csv/GridIoIT.groovy
# Conflicts:
#	CHANGELOG.md
#	src/main/java/edu/ie3/datamodel/io/processor/EntityProcessor.java
#	src/main/java/edu/ie3/datamodel/io/processor/Processor.java
#	src/main/java/edu/ie3/datamodel/io/processor/ProcessorProvider.java
#	src/main/java/edu/ie3/datamodel/io/processor/input/InputEntityProcessor.java
#	src/main/java/edu/ie3/datamodel/io/processor/result/ResultEntityProcessor.java
#	src/main/java/edu/ie3/datamodel/io/processor/timeseries/TimeSeriesProcessor.java
#	src/main/java/edu/ie3/datamodel/io/sink/CsvFileSink.java
#	src/main/java/edu/ie3/datamodel/io/sink/InfluxDbSink.java
#	src/test/groovy/edu/ie3/datamodel/io/sink/CsvFileSinkTest.groovy
#	src/test/groovy/edu/ie3/datamodel/io/source/csv/GridIoIT.groovy
@jo-bao jo-bao self-assigned this Nov 24, 2023
@jo-bao
Copy link
Contributor

jo-bao commented Nov 24, 2023

!test

@sebastian-peter
Copy link
Member

Closing this in favor of #767, which uses a fixed pattern and causes less "parameter pass-through"

@sebastian-peter sebastian-peter deleted the vb/#766_time_pattern branch December 6, 2023 15:01
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use the default time pattern for persisting time series keys instead of the toString method
4 participants