Skip to content

Conversation

ckittl
Copy link
Member

@ckittl ckittl commented Mar 26, 2020

Partly solves #73 (introduces an InputEntityProcessor for all InputEntity entities) and is based on #83.

when this one is merged, we still need processors for everything that has to do with input time series.

@ckittl ckittl requested a review from johanneshiry March 26, 2020 12:18
@ckittl ckittl self-assigned this Mar 26, 2020
@ckittl ckittl added the enhancement New feature or request label Mar 26, 2020
@ckittl ckittl added this to the Version 1.0 milestone Mar 26, 2020
@codecov
Copy link

codecov bot commented Mar 26, 2020

Codecov Report

Merging #86 into master will increase coverage by 0.64%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #86      +/-   ##
============================================
+ Coverage     60.96%   61.61%   +0.64%     
- Complexity      772      780       +8     
============================================
  Files           183      182       -1     
  Lines          3451     3413      -38     
  Branches        433      430       -3     
============================================
- Hits           2104     2103       -1     
+ Misses         1221     1179      -42     
- Partials        126      131       +5     
Impacted Files Coverage Δ Complexity Δ
...del/models/input/system/type/StorageTypeInput.java 53.33% <0.00%> (-1.84%) 8.00% <0.00%> (ø%)
.../ie3/datamodel/io/processor/ProcessorProvider.java 86.04% <0.00%> (-0.92%) 14.00% <0.00%> (-1.00%)
...du/ie3/datamodel/io/processor/EntityProcessor.java 87.50% <0.00%> (-0.62%) 40.00% <0.00%> (-5.00%)
...n/java/edu/ie3/datamodel/models/StandardUnits.java 95.65% <0.00%> (-0.10%) 1.00% <0.00%> (ø%)
...ava/edu/ie3/datamodel/models/input/AssetInput.java 57.89% <0.00%> (ø) 7.00% <0.00%> (+1.00%)
...e3/datamodel/models/input/connector/LineInput.java 38.46% <0.00%> (ø) 5.00% <0.00%> (ø%)
...3/datamodel/io/factory/input/NodeInputFactory.java 100.00% <0.00%> (ø) 3.00% <0.00%> (ø%)
...del/io/processor/result/ResultEntityProcessor.java 90.90% <0.00%> (ø) 6.00% <0.00%> (ø%)
...t/system/characteristic/EvCharacteristicInput.java 0.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...amodel/io/processor/input/AssetInputProcessor.java
... and 9 more

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 8247e76...b5d8e02. Read the comment docs.

@ckittl
Copy link
Member Author

ckittl commented Mar 27, 2020

!test

@ckittl ckittl mentioned this pull request Mar 27, 2020
ckittl added 2 commits March 30, 2020 17:19
…tProcessor

# Conflicts:
#	Jenkinsfile
#	src/test/groovy/edu/ie3/datamodel/io/factory/typeinput/SystemParticipantTypeInputFactoryTest.groovy
#	src/test/groovy/edu/ie3/datamodel/io/processor/ProcessorProviderTest.groovy
#	src/test/groovy/edu/ie3/datamodel/io/processor/input/GraphicInputProcessorTest.groovy
#	src/test/groovy/edu/ie3/datamodel/io/processor/input/InputEntityProcessorTest.groovy
#	src/test/groovy/edu/ie3/test/common/GridTestData.groovy
@ckittl
Copy link
Member Author

ckittl commented Mar 30, 2020

!test

@ckittl ckittl marked this pull request as ready for review March 30, 2020 15:23
@johanneshiry
Copy link
Member

!test

@johanneshiry
Copy link
Member

!test

Co-Authored-By: johanneshiry <johannes.hiry@tu-dortmund.de>
@ckittl
Copy link
Member Author

ckittl commented Mar 31, 2020

!test

@johanneshiry johanneshiry merged commit 5529c55 into master Mar 31, 2020
@johanneshiry johanneshiry deleted the ck/#73-inputProcessor branch March 31, 2020 11:11
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants