Skip to content
This repository was archived by the owner on Aug 8, 2024. It is now read-only.

SCF sampling code #4

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

SCF sampling code #4

wants to merge 31 commits into from

Conversation

nstarman
Copy link
Contributor

@nstarman nstarman commented Jul 25, 2021

Signed-off-by: Nathaniel Starkman (@nstarman) [email protected]

Description

Sample from an SCF Potential exactly or with an interpolation table

PR Checklist

  • Check out the contributing guidelines and code of conduct
  • Check out the contributing workflow ( for a practical example click here )
  • Give a detailed description of the PR above.
  • Document changes in the CHANGES.rst file. See existing changelog for examples.
  • Add tests, if applicable, to ensure code coverage never decreases.
  • Make sure the docs are up to date, if applicable, particularly the docstrings and RST files in docs folder.
  • Ensure linear history by rebasing, when requested by the maintainer.

nstarman added 14 commits July 29, 2021 12:38
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
@codecov
Copy link

codecov bot commented Jul 29, 2021

Codecov Report

Merging #4 (534562c) into main (36510aa) will decrease coverage by 3.96%.
The diff coverage is 96.02%.

❗ Current head 534562c differs from pull request most recent head d737299. Consider uploading reports for the commit d737299 to get more accurate results

@@             Coverage Diff             @@
##              main       #4      +/-   ##
===========================================
- Coverage   100.00%   96.03%   -3.97%     
===========================================
  Files            1        7       +6     
  Lines            1      454     +453     
===========================================
+ Hits             1      436     +435     
- Misses           0       18      +18     
Impacted Files Coverage Δ
sample_scf/core.py 81.81% <81.81%> (ø)
sample_scf/exact.py 90.08% <90.08%> (ø)
sample_scf/__init__.py 100.00% <100.00%> (ø)
sample_scf/_typing.py 100.00% <100.00%> (ø)
sample_scf/base.py 100.00% <100.00%> (ø)
sample_scf/interpolated.py 100.00% <100.00%> (ø)
sample_scf/utils.py 100.00% <100.00%> (ø)

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 36510aa...d737299. Read the comment docs.

Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
nstarman added 5 commits July 29, 2021 15:37
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
nstarman added 3 commits July 29, 2021 17:46
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: Nathaniel Starkman (@nstarman) <[email protected]>
Signed-off-by: nstarman <[email protected]>
Signed-off-by: nstarman <[email protected]>
Signed-off-by: nstarman <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant