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 Plummer potential #242

Merged
merged 15 commits into from
Jun 16, 2015
Merged

Add Plummer potential #242

merged 15 commits into from
Jun 16, 2015

Conversation

jobovy
Copy link
Owner

@jobovy jobovy commented Jun 15, 2015

To do items:

  • Implement the potential, its forces, and derivatives
  • C implementation
  • Documentation
  • Add to changelog
  • Add nemo functions and list somewhere in docs
  • Fully test:
    • Derivatives (automatic)
    • C integration (automatic)
    • Actions
    • NEMO

@jobovy jobovy self-assigned this Jun 15, 2015
@jobovy jobovy added this to the v1.1 milestone Jun 15, 2015
@codecov-io
Copy link

Current coverage is 99.66%

Merging #242 into dev-v1.1 will change coverage by 0% by 006e0bd

Coverage Diff

@@            dev-v1.1    #242   diff @@
========================================
  Files             97      99     +2
  Stmts          14713   14796    +83
  Branches           0       0       
  Methods            0       0       
========================================
+ Hit            14663   14746    +83
  Partial            0       0       
  Missed            50      50       

Powered by Codecov

jobovy added a commit that referenced this pull request Jun 16, 2015
@jobovy jobovy merged commit 6cebe94 into dev-v1.1 Jun 16, 2015
@jobovy jobovy deleted the plummer branch June 16, 2015 01:13
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.

2 participants