add simple_plotly function for pandapipes #430
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* only tested with gas grids * some components are still missing (heat exchanger, compressor...)
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #430 +/- ##
===========================================
- Coverage 87.91% 86.82% -1.09%
===========================================
Files 79 81 +2
Lines 4717 4776 +59
===========================================
Hits 4147 4147
- Misses 570 629 +59 ☔ View full report in Codecov by Sentry. |
Hotfix 0.8.2
Hotfix 0.8.3
In file_io: do not pass object_hook to JSON Decoder in json.loads, but registry_class. Please do not merge before pandapower has been updated to process this change! (cf. e2nIEE/pandapower#1827)
Load JSON string simplified
* For the test job, make sure that the repo is not checked out and the only available pandapipes installation is the one from pip * Update the versions of the Actions Co-authored-by: jkisse <[email protected]>
Hotfix for nx_graph creation of networks with flow controllers, as the keywords concerning them were not correctly considered within the function (include_flow_controls could not be set to False).
Flow control is considered in nx_graph creation
install Numba only for Python < 3.11
* enable multiple creation of heat exchanger * adapted changelog and added doc for multiple heat exchanger
Hotfix 0.8.4
* add Python 3.11 to Release workflow * change Python Version of docs_check and linting to 3.10
remove direct import from toolbox file
* remove Python 3.7 from test workflow * update CHANGELOG.rst
|
rename to simple_plotly_gas |
- 2 function calls to connectivity check and pit reduction in case of "all" mode - result extraction from active pit to pit done after both calculations separately - all result extraction functions (also for all components) must consider that the connectivity lookups now differ between hydraulics and thermal calculation
readthedocs.yml deprecation
Biomethane (bis)
…pipes were considered. Adapted it accordingly that any branch component is considered.
…andapipes into global_thermal
calling 'calculate_derivatives_thermal' once
Bugfix/generic geodata
|
Traces for compressors and valves added (as lines with dedicated colors) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Like
simple_plotlyfrom pandapower but for pandapipes (interactive plotting function).So far, only tested with gas grids.
Some components are still missing and the infofunction could be improved.
For every component, a
create_..._tracefunction should be implemented. Different marker symbols can be used. Plotly provides a bunch of pre-defined symbols (listed here). If you have suggestions, which symbols are suitable for specific components, please let us know.marker: bowtie)Contributions welcome!