-
Notifications
You must be signed in to change notification settings - Fork 18
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
Enhancement: ContactTrajectory class (with movies!) #53
Comments
+1 for the idea.
Which one do you expect to be cheaper, using the methods or the |
Not sure what you mean here... these calculate different things. If you have Those methods were designed specifically for doing this sort of windowing of a contact frequency, so those are the things to use. |
Ah I understand now :) thanks for the explanation |
@dwhswenson was there any issue left here after #81 ? |
#81 doesn't include movies, just the |
Good to know, was just checking in on the current state of things |
It might be nice to add a
ContactTrajectory
class. This would store a separate (single frame) contact map for each time slice. Then you could do interesting things like create aContactFrequency
based on a set of frames determined after the fact. You could also create movies of how the contact map evolves in time (these should be based on some windowing intoContactFrequency
instances; the window can be 1 if the user wants).Relevant matplotlib docs on animation:
Animations that could be generated in real time would be impressive, but I'd be just as happy with something that pre-generates the images and makes a movie out of them, like
ArtistAnimation
does.Note that, to use a rolling window to generate the contact frequency plots for each movie frame, we already have
add_contact_frequency
andsubtract_contact_frequency
methods. These affect the absolute histogram count, unlikeContactDifference
, which compares the after-normalization (to number of frames) contact frequencies. We'd still output a separate plot at each time step, but the underlying data would be cheaper to calculate.The text was updated successfully, but these errors were encountered: