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.

Sign up for GitHub

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

Jump to bottom

Add tests for using Bayes Ephem with PINT. #250

Open
AaronDJohnson opened this issue Feb 5, 2021 · 0 comments
Open

Add tests for using Bayes Ephem with PINT. #250

AaronDJohnson opened this issue Feb 5, 2021 · 0 comments
Milestone
release-3.1

Comments

@AaronDJohnson
Copy link
Collaborator

Currently deterministic_signals.py PINT test only makes sure that you are not using PINT. Bayes Ephem is now functional with PINT, and the tests need to be updated with checks similar to what is already done with TEMPO2 + BE.

PR #249 removes 5 lines of code that stop anyone from using PINT with BE.

Removal of these lines results in
FAILED tests/test_deterministic_signals.py::TestDeterministicSignalsPint::test_physical_ephem_model
@AaronDJohnson AaronDJohnson changed the title Add tests for Bayes Ephem in PINT. Add tests for using Bayes Ephem with PINT. Feb 5, 2021
@vallis vallis added this to the release-3.1 milestone Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
release-3.1
Development

No branches or pull requests
2 participants
@vallis @AaronDJohnson