Skip to content

Improvements to unwrapping/wrapping for trajectory processing in the BilayerAnalyzer class. #25

@blakeaw

Description

@blakeaw

The BilayerAnalyzer class assumes that the coordinates of the input trajectory are wrapped by atom and therefore automatically unwraps the coordinates by atom. This can fail when the first frame is not a good reference for unwrapping. This needs to be upgraded to allow for more options of processing the trajectory, such as the opposite scenario where the input trajectory is unwrapped and needs to be wrapped. It would also be good to allow the user to be able to turn off wrapping/unwrapping.
For the currently assumed case (atom-wise wrapped coordinates), it would probably be good to add in a separate input setting for a reference structure; this could default to the first frame when unwrapping used.

A possible long term solution might be to add a transformations protocol (e.g wrapping/unwrapping, structure alignment) that can be set by the user (with proper defaults and conflict resolution based on the analyses being implemented) that is similar to the suggestion in MDAnalysis/mdanalysis#786

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementRequest or suggestion for an enhancement to current features or api.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions