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

Add pytest to sanity check basic example #11

Open
11 tasks done
joshkamm opened this issue Oct 30, 2024 · 0 comments · May be fixed by #21
Open
11 tasks done

Add pytest to sanity check basic example #11

joshkamm opened this issue Oct 30, 2024 · 0 comments · May be fixed by #21
Assignees

Comments

@joshkamm
Copy link
Member

joshkamm commented Oct 30, 2024

  • create new branch
  • add pytest task, feature, and dependency in pixi.toml
  • use task dependency to run test so pytest can assume the output already exists and just read it
  • add pytest to pixi start task
  • use the test function to check that the output integrals are similar to a pre-saved sample output
  • try relative tolerance 10^-6 and absolute 10^-6
  • move python test file
  • add pycache to .gitignore
  • ask which matrices I should test - Paul said A, S, En, T, pVp, and Ciap are all good
  • figure out how to test multiple matrices in single file for SENT
  • create pr
@joshkamm joshkamm self-assigned this Oct 30, 2024
joshkamm added a commit that referenced this issue Nov 6, 2024
joshkamm added a commit that referenced this issue Nov 7, 2024
add numpy to pixi pytest environment
joshkamm added a commit that referenced this issue Nov 15, 2024
joshkamm added a commit that referenced this issue Nov 15, 2024
move location of pytest file from specific example to examples folder
add pycache to gitignore
@joshkamm joshkamm linked a pull request Dec 14, 2024 that will close this issue
@joshkamm joshkamm linked a pull request Dec 14, 2024 that will close this issue
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 a pull request may close this issue.

1 participant