Skip to content

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