search

California-Planet-Search / radvel

General Toolkit for Modeling Radial Velocity Data
http://radvel.readthedocs.io
MIT License
57 stars 52 forks source link

Cannot run radvel report in tutorial without first doing a model comparison #341

Closed radicamc closed 3 years ago

radicamc commented 3 years ago

Hello, I'm working through the initial radvel tutorial and have run into a potential bug when trying to call $ radvel report -s /path/to/radvel/example_planets/HD164922.py. Here is the detail of the error:

File "/Users/michaelradica/.anaconda3/envs/astroconda/bin/radvel", line 8, in sys.exit(main()) File "/Users/michaelradica/.anaconda3/envs/astroconda/lib/python3.7/site-packages/radvel/cli.py", line 197, in main args.func(args) File "/Users/michaelradica/.anaconda3/envs/astroconda/lib/python3.7/site-packages/radvel/driver.py", line 586, in report compstats = eval(status.get('ic_compare', 'ic')) File "/Users/michaelradica/.anaconda3/envs/astroconda/lib/python3.7/configparser.py", line 780, in get d = self._unify_values(section, vars) File "/Users/michaelradica/.anaconda3/envs/astroconda/lib/python3.7/configparser.py", line 1146, in _unify_values raise NoSectionError(section) from None configparser.NoSectionError: No section: 'ic_compare'

it seems to be that the except KeyError test on line 587 in driver.py is not catching the error thrown by the above try if no model comparison has been preformed. Indeed, if I proceed and run a model comparison with $ radvel ic -t nplanets e trend -s /path/to/radvel/example_planets/HD164922.py before radvel report, everything seems fine.

bjfultn commented 3 years ago

Thank you for this detailed bug report. This will be fixed in v1.4.4