Skip to content

Add unwrap to radius_of_gyration#2305

Closed
NinadBhat wants to merge 1 commit intoMDAnalysis:developfrom
NinadBhat:test_rog
Closed

Add unwrap to radius_of_gyration#2305
NinadBhat wants to merge 1 commit intoMDAnalysis:developfrom
NinadBhat:test_rog

Conversation

@NinadBhat
Copy link
Copy Markdown
Contributor

Fixes #1760

Changes made in this Pull Request:

  • Adds unwrap keyword to radius of gyration

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

@NinadBhat NinadBhat mentioned this pull request Jul 30, 2019
4 tasks
@NinadBhat
Copy link
Copy Markdown
Contributor Author

@orbeckst @richardjgowers can you kindly rerun the tests here?

@zemanj
Copy link
Copy Markdown
Member

zemanj commented Aug 5, 2019

@NinadBhat You can trigger a new CI test run yourself by simply closing and reopening the PR!

@NinadBhat NinadBhat closed this Aug 5, 2019
@NinadBhat NinadBhat reopened this Aug 5, 2019
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 5, 2019

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.76%. Comparing base (89a9514) to head (150590f).
Report is 1703 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2305      +/-   ##
===========================================
+ Coverage    89.63%   89.76%   +0.12%     
===========================================
  Files          173      173              
  Lines        21498    21592      +94     
  Branches      2801     2832      +31     
===========================================
+ Hits         19269    19381     +112     
+ Misses        1628     1615      -13     
+ Partials       601      596       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@NinadBhat
Copy link
Copy Markdown
Contributor Author

@zemanj Thanks!

@NinadBhat
Copy link
Copy Markdown
Contributor Author

@richardjgowers can you kindly review this? I have answered some of your questions at #2299.

@IAlibay IAlibay added the close? Evaluate if issue/PR is stale and can be closed. label May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

close? Evaluate if issue/PR is stale and can be closed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Radius of Gyration needs wrap option

3 participants