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

fix: create_isobar_topologies only requires n_final#500

Merged
redeboer merged 1 commit intomasterfrom
fix-create_isobar_topologies
Feb 25, 2021
Merged

fix: create_isobar_topologies only requires n_final#500
redeboer merged 1 commit intomasterfrom
fix-create_isobar_topologies

Conversation

@redeboer
Copy link
Copy Markdown
Member

create_isobar_topologies now only needs a number of final states: it is assumed that the number of initial states is one.

@redeboer redeboer added the ⚠️ Interface Changes to the interface label Feb 25, 2021
@redeboer redeboer self-assigned this Feb 25, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2021

Codecov Report

Merging #500 (48b0481) into master (ebaa93b) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #500      +/-   ##
==========================================
- Coverage   89.21%   89.14%   -0.07%     
==========================================
  Files          22       22              
  Lines        3338     3336       -2     
  Branches      845      844       -1     
==========================================
- Hits         2978     2974       -4     
- Misses        195      196       +1     
- Partials      165      166       +1     
Flag Coverage Δ
unittests 89.14% <100.00%> (-0.07%) ⬇️

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

Impacted Files Coverage Δ
src/expertsystem/reaction/topology.py 90.23% <ø> (-0.07%) ⬇️
src/expertsystem/reaction/__init__.py 83.57% <100.00%> (ø)
src/expertsystem/reaction/conservation_rules.py 91.56% <0.00%> (-0.59%) ⬇️

@redeboer redeboer merged commit 6e00fbb into master Feb 25, 2021
@redeboer redeboer deleted the fix-create_isobar_topologies branch February 25, 2021 11:35
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