Skip to content
This repository was archived by the owner on Oct 11, 2021. It is now read-only.

feat: allow hiding initial state and resonance ids#546

Merged
redeboer merged 4 commits intomasterfrom
only-final-state-ids
Mar 28, 2021
Merged

feat: allow hiding initial state and resonance ids#546
redeboer merged 4 commits intomasterfrom
only-final-state-ids

Conversation

@redeboer
Copy link
Copy Markdown
Member

@redeboer redeboer commented Mar 28, 2021

Argument render_edge_id in asdot has been split into render_final_state_id, render_resonance_id, and render_initial_state_id.

Edges IDs of initial states and resonances are now hidden by default. They can be shown with the arguments of asdot:

import expertsystem as es

es.io.asdot(
    graphs,
    render_node=True,
    render_final_state_id=True,
    render_resonance_id=True,
    render_initial_state_id=True,
    collapse_graphs=True,
)

Default view of a StateTransitionGraph:

@redeboer redeboer added the 💡 Feature New feature added to the package label Mar 28, 2021
@redeboer redeboer added this to the Release 0.7.2 milestone Mar 28, 2021
@redeboer redeboer self-assigned this Mar 28, 2021
@redeboer redeboer added ⚠️ Interface Changes to the interface and removed 💡 Feature New feature added to the package labels Mar 28, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 28, 2021

Codecov Report

Merging #546 (6e310ac) into master (e753853) will increase coverage by 0.12%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #546      +/-   ##
==========================================
+ Coverage   86.19%   86.31%   +0.12%     
==========================================
  Files          22       22              
  Lines        3628     3632       +4     
  Branches      884      885       +1     
==========================================
+ Hits         3127     3135       +8     
+ Misses        322      320       -2     
+ Partials      179      177       -2     
Flag Coverage Δ
unittests 86.31% <100.00%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/expertsystem/io/__init__.py 88.09% <ø> (ø)
src/expertsystem/io/_dot.py 75.88% <100.00%> (+1.78%) ⬆️
src/expertsystem/amplitude/helicity.py 86.29% <0.00%> (+0.62%) ⬆️

@redeboer redeboer merged commit 2112fd4 into master Mar 28, 2021
@redeboer redeboer deleted the only-final-state-ids branch March 28, 2021 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚠️ Interface Changes to the interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant