Skip to content

Commit

Permalink
Merge pull request #354 from sblunt/rebound-bugfix
Browse files Browse the repository at this point in the history
- Replace deprecated calculate_com function to get unit tests to pass
- CI tests now pass, coveralls works correctly, and jupyter nbs are run as part of CI
  • Loading branch information
sblunt authored Jan 12, 2024
2 parents 3424fcc + dc93e0c commit 83e7976
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/run-notebooks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,5 +32,5 @@ jobs:
python -m pip install . --no-build-isolation
- name: Run tutorial notebooks
run: |
py.test --nbmake docs/tutorials/*.ipynb
py.test --nbmake --nbmake-timeout=3000 docs/tutorials/*.ipynb
2 changes: 1 addition & 1 deletion orbitize/nbody.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def calc_orbit(
for j,t in enumerate(te):
sim.integrate(t/365.25)
#for the star and each planet in each epoch denoted by j,t find the RA, Dec, and RV
com = sim.calculate_com()
com = sim.com()
ra_reb[j,0] = -(ps[0].x-com.x)# ra is negative x
dec_reb[j,0] = ps[0].y-com.y
vz[j,0] = ps[0].vz
Expand Down

0 comments on commit 83e7976

Please sign in to comment.