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

Adding tidynamics to requirements #4853

Closed
wants to merge 2 commits into from

Conversation

tulga-rdn
Copy link

@tulga-rdn tulga-rdn commented Dec 20, 2024

Fixes #4852

Changes made in this Pull Request:

  • Added tidynamics to requirements.txt
  • Removed the correspodning error in /package/MDAnalysis/analysis/msd.py

PR Checklist

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

Developers certificate of origin


📚 Documentation preview 📚: https://mdanalysis--4853.org.readthedocs.build/en/4853/

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello there first time contributor! Welcome to the MDAnalysis community! We ask that all contributors abide by our Code of Conduct and that first time contributors introduce themselves on GitHub Discussions so we can get to know you. You can learn more about participating here. Please also add yourself to package/AUTHORS as part of this PR.

@RMeli RMeli self-assigned this Dec 20, 2024
@RMeli RMeli added the close? Evaluate if issue/PR is stale and can be closed. label Dec 20, 2024
@tulga-rdn tulga-rdn closed this Dec 20, 2024
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.

tidynamics required by FFT in EinsteinMSD but not in requirements
2 participants