Closed marjanalbooyeh closed 1 year ago
Merging #95 (c01bb02) into master (ea09fde) will increase coverage by
0.15%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #95 +/- ##
==========================================
+ Coverage 92.14% 92.29% +0.15%
==========================================
Files 7 7
Lines 751 766 +15
==========================================
+ Hits 692 707 +15
Misses 59 59
Impacted Files | Coverage Δ | |
---|---|---|
polybinder/system.py | 95.67% <100.00%> (+0.16%) |
:arrow_up: |
Thanks for the PR @marjanAlbouye! Looks good to me. I just made one small change to get the line lengths under 80.
This PR saves the ParmEd structure built during system initialization to a file. After saving the ParmEd to the file, for any future initialization of the same system, ParmEd object is loaded from file instead of building it from scratch. This will save a significant amount of time, which makes running simulations with different parameters (density, temperature, ...) much faster.
Details:
save_parmed=False
inInitializer
class.pmd_structures
, which can be changed by settingparmed_dir
argument inInitializer
class.pickle
. The reason I did not use ParmEd save function is because it loses all the connections (bonds, angles and dihedrals)