-
Notifications
You must be signed in to change notification settings - Fork 673
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
deprecate hydrogen bond analysis in water bridge analysis #2913
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
2a1a092
depricate hydrogen bond analysis in water bridge
68584d9
PEP8
eeedf71
PEP8
b007a63
PEP8
267ef75
PEP8
306fde1
fix import
e521435
Update CHANGELOG
c33d2dc
DeprecationWarning
ce5c552
PEP8
54efe5e
Update package/CHANGELOG
xiki-tempula e17e742
Update package/MDAnalysis/analysis/hbonds/wbridge_analysis.py
xiki-tempula 728f154
Update package/MDAnalysis/analysis/hydrogenbonds/__init__.py
xiki-tempula 8658ab3
Update wbridge_analysis.py
xiki-tempula 7fad43d
warning test
689035b
Merge branch 'develop' into develop
xiki-tempula 05ef615
PEP8
1d88243
Merge branch 'develop' of https://github.com/xiki-tempula/mdanalysis …
d7305f7
PEP8
01f0297
warning
6c0f56b
PEP8
e852f7a
catch warning
946a367
arrange order
6f4517c
fix loading issue
11258e7
Merge branch 'develop' of https://github.com/MDAnalysis/mdanalysis in…
da262bf
remove the file
53a1913
fix test
fa1d187
Update wbridge_analysis.rst
2cb9e03
Merge branch 'develop' of https://github.com/MDAnalysis/mdanalysis in…
d7ce7e5
fix doc string waring
xiki-tempula e92d8ce
fix doc
xiki-tempula File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically new changes go to the top of the list, but I'm willing to let that go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I appreciate your attention to detail.
I merged the whole thing nevertheless, as this has been sitting for way too long.