-
Notifications
You must be signed in to change notification settings - Fork 7
Misc #134
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
Misc #134
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.
Copilot reviewed 48 out of 50 changed files in this pull request and generated 2 comments.
Files not reviewed (2)
- docs/source/api.rst: Language not supported
- docs/source/release-notes.rst: Language not supported
Comments suppressed due to low confidence (1)
ams/interface.py:116
- The removal of the sync_adsys() call in the to_andes_pflow() function might lead to unsynchronized parameter values between AMS and ANDES systems. Please verify whether the synchronization step should be retained.
return adsys
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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.
Copilot reviewed 53 out of 53 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
ams/interface.py:108
- Typo in parameter 'no_outpu'; it should be 'no_output' to ensure that the andes_System function receives the correct argument.
adsys = andes_System(no_outpu=no_output, default_config=default_config, **kwargs)
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferences |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #134 +/- ##
===========================================
+ Coverage 78.59% 78.65% +0.05%
===========================================
Files 71 71
Lines 5967 5959 -8
===========================================
- Hits 4690 4687 -3
+ Misses 1277 1272 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Area
fromZone
for zonal calculationStaticGen
andStaticLoad
witharea
pjm5bus_demo.xlsx
as a all-inclusive caseMatProcessor
in the API documentationSystem.setup()
interface._to_andes_pflow
public