Skip to content
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

Allow defining a TS species simply with .xyz guesses #124

Merged
merged 2 commits into from
Apr 30, 2019
Merged

Allow defining a TS species simply with .xyz guesses #124

merged 2 commits into from
Apr 30, 2019

Conversation

alongd
Copy link
Member

@alongd alongd commented Apr 30, 2019

instead of defining TSGuess objects

alongd added 2 commits April 29, 2019 21:07
Can be calculated without a reaction context
instead of defining TSGuess objects
@alongd alongd merged commit 5b39b88 into master Apr 30, 2019
@alongd alongd deleted the tsg branch April 30, 2019 01:42
@codecov
Copy link

codecov bot commented Apr 30, 2019

Codecov Report

Merging #124 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
- Coverage   40.48%   40.43%   -0.05%     
==========================================
  Files          22       22              
  Lines        5748     5755       +7     
  Branches     1527     1530       +3     
==========================================
  Hits         2327     2327              
- Misses       3041     3046       +5     
- Partials      380      382       +2
Impacted Files Coverage Δ
arc/species/species.py 58.15% <0%> (-0.4%) ⬇️
arc/scheduler.py 20.23% <0%> (ø) ⬆️
arc/reaction.py 41.69% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 355f6a3...6bd5345. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant