-
Notifications
You must be signed in to change notification settings - Fork 1
Refactor entropy calculations: switch to per-timestep computation with molecule-type grouping #126
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
Merged
harryswift01
merged 26 commits into
main
from
30-averaging-over-molecules-of-the-same-type
Aug 4, 2025
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
0f8b976
Switching loops to loop over timesteps first and then molecules when …
skfegan d314f53
applying pre-commit hooks and linting for code quality
harryswift01 ced888d
Refactor matrix accumulation to handle shape mismatches with zero-pad…
harryswift01 9514e7e
Merge branch 'main' of https://github.com/CCPBioSim/CodeEntropy into …
harryswift01 aa42de4
Refactor get_beads to remove padding and simplify force/torque matrix…
harryswift01 57250c4
add in averaging over the `number_frames` for each of the forces and …
harryswift01 909772d
Fix timestep iteration by applying correct slicing to ensure the corr…
harryswift01 a267f47
Refactor repeated for-loops into a unified loop to eliminate redundancy
harryswift01 3214289
Fix indentation and matrix key usage in `_process_united_atom_entropy`
harryswift01 0ac02af
removal of duplicated timestep slice within the united atom level cal…
harryswift01 d23ec92
Ensuring if the united atom isn't bonded, the axis are reproduceable …
harryswift01 ea381c0
filtering matrices to avoid zero eigenvalues
skfegan e545284
Merge branch '30-averaging-over-molecules-of-the-same-type' of https:…
harryswift01 3e17391
correcting conformational state string
skfegan c144f98
Merge branch '30-averaging-over-molecules-of-the-same-type' of https:…
harryswift01 4183498
run pre-commit on all files to ensure code consistency
harryswift01 5498916
Refactor timestep entropy computation workflow into modular, maintain…
harryswift01 d6cbdc1
Further refinements to the per timestep refactor, relocating function…
harryswift01 80ced0c
fixing and adding additional test cases for the per-timestep switch over
harryswift01 218616c
separating the building of conformational states from the covariance …
skfegan fa65669
adding averaging
skfegan 3c38e72
applying pre-commit hooks and linting for code quality
harryswift01 0f87a04
fix timestep slicing issue within `assign_conformation`
harryswift01 fcf83ef
remove unused `number_molecules` from `build_conformational_states` f…
harryswift01 83d827d
fixing and adding additional test cases for the grouping by molecule …
harryswift01 fe089bd
fixing comments within the testing files
harryswift01 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 hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.