search

rmcgibbo / openmm-cmd

OpenMM Command Line Application
7 stars 3 forks source link

inline comments in config file #10

Closed rmcgibbo closed 11 years ago

rmcgibbo commented 11 years ago 
$ python ~/projects/openmm-cmd/openmm --config config.ini
[OpenMM] ERROR | The 'traj_freq' trait of the simulation section instance must be an int, but a value of '1000  # 2 picoseconds' <type 'str'> was specified.

To see all available configurables, use `--help-all`
rmcgibbo commented 11 years ago

the comment should be filtered out before the typechecking/casting to int.

leeping commented 11 years ago

Hi Robert,

I found this problem as well. It seems the ConfigParser from the Python Standard Library should be taking care of the comments, do you know why it's not?

rmcgibbo commented 11 years ago

For some reason the stdlib doesn't do this. But its fixed now.