Skip to content

added CSEP_tests.ipynb #173

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

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Conversation

kirstybayliss
Copy link
Contributor

added test notebook with theory and examples for pycsep tests

added test notebook
@codecov-commenter
Copy link

Codecov Report

Merging #173 (dbc4cb0) into master (a521db5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #173   +/-   ##
=======================================
  Coverage   56.09%   56.09%           
=======================================
  Files          20       20           
  Lines        3264     3264           
  Branches      519      519           
=======================================
  Hits         1831     1831           
  Misses       1314     1314           
  Partials      119      119           

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 a521db5...dbc4cb0. Read the comment docs.

@wsavran wsavran merged commit af298a5 into SCECcode:master Jan 14, 2022
@mherrmann3
Copy link
Contributor

We should add that this notebook can also be found with much better formatting in /docs/getting_started/theory.rst (#171) - and vice versa: mention in the docs that the contents can be downloaded as Jupyter notebook (https://github.com/SCECcode/pycsep/blob/master/notebooks/CSEP_tests.ipynb).
Alternatively, we could just drop it, but this would remove the opportunity of downloading the notebook and let users play with it.

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

Successfully merging this pull request may close these issues.

4 participants