Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • S spinifel
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 35
    • Issues 35
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 7
    • Merge requests 7
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • MTIP
  • spinifel
  • Merge requests
  • !51

Reference volume from MRC or PDB file

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Ariana Peck requested to merge ap_fsc_mod into development Oct 22, 2022
  • Overview 3
  • Commits 14
  • Changes 12

To address a numba-related incompatibility on Crusher, the code was adjusted so that either a PDB or MRC file can be provided as the reference. If the latter, the density array is simply loaded rather than computed from a coordinates file using skopi, which depends on numba. The relevant entry in the toml file has been changed from pdb_path to ref_path. This was tested on Cori GPU as follows:

srun -n 1 -G 1 python -m spinifel --default-settings=cgpu_mpi.toml --mode=mpi

with the chk_convergence flag set to true.

@yoon82, is there a Crusher-specific toml file? I wasn't sure which toml files should use an MRC rather than a PDB file; for now, I've only updated the cgpu_mpi.toml to use the former.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: ap_fsc_mod