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

Buffered, on-demand reading of time series #421

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

ckittl
Copy link
Member

@ckittl ckittl commented Sep 2, 2021

Closes #420

@ckittl ckittl added the enhancement New feature or request label Sep 2, 2021
@ckittl ckittl added this to the Version 2.1 milestone Sep 2, 2021
@ckittl ckittl requested a review from a team September 2, 2021 14:13
@ckittl ckittl self-assigned this Sep 2, 2021
@ckittl ckittl added this to In progress in TaskBoard via automation Sep 2, 2021
@codecov
Copy link

codecov bot commented Sep 2, 2021

Codecov Report

Merging #421 (a355423) into dev (f922492) will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #421      +/-   ##
============================================
- Coverage     78.92%   78.86%   -0.07%     
- Complexity     2106     2138      +32     
============================================
  Files           259      261       +2     
  Lines          8253     8357     +104     
  Branches        775      783       +8     
============================================
+ Hits           6514     6591      +77     
- Misses         1346     1363      +17     
- Partials        393      403      +10     
Impacted Files Coverage Δ
...in/java/edu/ie3/datamodel/io/sink/CsvFileSink.java 73.21% <0.00%> (-2.57%) ⬇️
.../ie3/datamodel/io/connectors/CsvFileConnector.java 78.18% <0.00%> (-1.96%) ⬇️
...3/datamodel/io/source/csv/CsvTimeSeriesSource.java 95.91% <0.00%> (-0.17%) ⬇️
.../edu/ie3/datamodel/io/source/TimeSeriesSource.java 100.00% <0.00%> (ø)
...del/io/source/csv/CsvWindowedTimeSeriesSource.java 78.31% <0.00%> (ø)
...datamodel/io/naming/DefaultDirectoryHierarchy.java 93.96% <0.00%> (+0.86%) ⬆️
...u/ie3/datamodel/models/value/EnergyPriceValue.java 60.00% <0.00%> (+20.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 124d2fc...a355423. Read the comment docs.

@johanneshiry johanneshiry modified the milestones: Version 2.1, Version 2.2 Jan 5, 2022
@sebastian-peter sebastian-peter modified the milestones: Version 2.2, Version 3.0 Feb 23, 2022
@sebastian-peter sebastian-peter modified the milestones: Version 4.1, Version 4.2 Nov 2, 2023
@sebastian-peter sebastian-peter modified the milestones: Version 4.2, Version 5.0 Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
TaskBoard
  
In progress
Development

Successfully merging this pull request may close these issues.

CSV time series source with window buffer
3 participants