Conversation
… same lipid type - Added files in tests/data/prestin - Updated datafiles.py
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
orbeckst
left a comment
There was a problem hiding this comment.
You accidentally checked in a build directory: get rid of it git rm -r build
|
@rjoshi44 can you please remove the build directory from your PR, see above for how. thanks! |
orbeckst
left a comment
There was a problem hiding this comment.
Thanks for the cleanup @rjoshi44 . Can you please move the files so that they correspond to the file paths in datafiles.py?
Check that you can do from python
import os.path
from basicrta.tests import datafiles
assert os.path.exists(datafiles.PDB_PAPE_1)
...There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
The pdbs and trajectories should be inside the tests/data/prestin directory and not in subdirectories (see datafiles.py).
There was a problem hiding this comment.
This file is 18 MB – why is this so much bigger? Is there a way to make it smaller??
There was a problem hiding this comment.
I have removed subdirectories and put all the pdb and xtc files in tests/data/prestin.
There was a problem hiding this comment.
The file pape3_20ns.xtc was larger than the others because its output was saved every 10 ps. I reduced the size for this file.
…ta/prestin reduced the size of pape3_20ns.xtc
|
@rjoshi44 It looks like you added only |
Fixes #42
Changes made in this Pull Request:
PR Checklist