You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Couldn't the same purpose be achieved by a more portable test? Namely try to open the file, and if not good() we give up.
Second question: will this play nicely with the input file caching in GROMACS? At mdrun time, the input files may not exist in the filesystem.
Both good points. In that case, rather than reinventing the wheel or making the tests more complex for GROMACS, we could perhaps recognize that load_coords() is really intended for PDB files, and make its error messages more explicit that way.
This would also help in the case where people try using a PDB file in NAMD or VMD and then suddenly it doesn't work in GROMACS...
Both good points. In that case, rather than reinventing the wheel or making the tests more complex for GROMACS, we could perhaps recognize that load_coords() is really intended for PDB files, and make its error messages more explicit that way.
When loading a file that isn't XYZ the first branch of the conditional below fails, falling back to the PDB reader:
colvars/src/colvarmodule.cpp
Lines 2159 to 2171 in 8eb35e7
In GROMACS the resulting error message can be confusing.
It may be clearer to check first if the file is present and readable before discussing its format.
The text was updated successfully, but these errors were encountered: