-
Notifications
You must be signed in to change notification settings - Fork 5
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
DBFS with multiple connections between sub grids #25
Conversation
between agents in DBFS algorithm when there are multiple connections between two adjacent GridAgents. Co-authored-by: sebastian-peter <sebastian.peter@tu-dortmund.de>
Codecov Report
@@ Coverage Diff @@
## dev #25 +/- ##
==========================================
+ Coverage 79.78% 79.84% +0.05%
==========================================
Files 159 159
Lines 5838 5869 +31
Branches 80 81 +1
==========================================
+ Hits 4658 4686 +28
- Misses 1180 1183 +3
Continue to review full report at Codecov.
|
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.
Some small things I noticed
…nections # Conflicts: # CHANGELOG.md
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
to inferior GridAgents
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
and relevant places of usage
c650307
to
73dcc5a
Compare
I guess #131 should be merged before this PR. |
…nections # Conflicts: # CHANGELOG.md
# Conflicts: # CHANGELOG.md # src/main/scala/edu/ie3/simona/config/ConfigFailFast.scala # src/main/scala/edu/ie3/simona/config/RefSystemParser.scala # src/main/scala/edu/ie3/simona/config/SimonaConfig.scala # src/test/scala/edu/ie3/simona/config/ConfigFailFastSpec.scala # src/test/scala/edu/ie3/simona/config/RefSystemParserSpec.scala
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Nice, I just have minor things which might be taken into account.
Co-authored-by: Daniel Feismann <98817556+danielfeismann@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
Analysis Details1 IssueCoverage and DuplicationsProject ID: edu.ie3:simona |
# Conflicts: # CHANGELOG.md # src/main/scala/edu/ie3/simona/agent/grid/PowerFlowSupport.scala # src/test/scala/edu/ie3/simona/agent/grid/DBFSAlgorithmCenGridSpec.scala
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.
Thanks for your work 🙃
|
||
implicit class RichUnit[Q <: Quantity[Q]]( | ||
private val unit: measure.Unit[Q] | ||
) extends AnyVal { | ||
|
||
/** Transform some power unit to given unit with the same prefix | ||
* @param targetUnit | ||
* the target system unit | ||
* @return | ||
* this unit converted to given | ||
*/ | ||
def toEquivalentIn(targetUnit: measure.Unit[Q]): measure.Unit[Q] = | ||
targetUnit.transform(unit.getConverterTo(unit.getSystemUnit)) | ||
} |
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.
I think it makes sense to transfer this class (as well as the RichQuantity) into the PowerSystemUtils as it might come in handy for other projects as well and we already have som Quantity related utils there.
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.
I'm fine with doing that at some point in time. I'd leave it in SIMONA until it's available in PSU.
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.
ie3-institute/PowerSystemUtils#289
If you want to make a quick review we can add it while were at it
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.
Ok, that was really quick. I removed RichUnit
and RichQuantity
here and replaced all calls with ones to the equivalents in PSU
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.
Great!
Resolves #22
Resolves #129
Edit (SP): Please review #266 first!
Edit (SP): Test grid for
DBFSAlgorithmCenGridSpec
adapted to only have three inferior subgrids, one of which having two connections to the central subgrid