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.

Sign up for GitHub

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

Jump to bottom

[TODO] Split analysis.py #96

Open
slochower opened this issue Feb 28, 2019 · 2 comments
Open

[TODO] Split analysis.py #96

slochower opened this issue Feb 28, 2019 · 2 comments
Assignees
@slochower @jeff231li
Labels
enhancement

Comments

@slochower
Copy link
Member

It's getting unwieldy and not everything needs to be in the same class.
@slochower slochower self-assigned this Feb 28, 2019
@slochower slochower changed the title Split analysis.py [TODO] Split analysis.py Mar 1, 2019
@jeff231li
Copy link
Collaborator

Maybe to improve the speed we could Cythonize some of the code and/or parallelize. Analyzing large trajectory files and convergence analysis with fractions takes a very long time to complete.

@slochower
Copy link
Member Author

slochower commented Oct 27, 2020 •
edited
Loading

That's a good idea. I think some of the for loops can probably be avoided, and if not, can be coded in C since they're generally just looping through frames or coordinates.

@jeff231li jeff231li self-assigned this Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slochower slochower

@jeff231li jeff231li

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@slochower @jeff231li