search

California-Planet-Search / radvel

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

Multipanel plot fix #371

Closed sealauren closed 11 months ago

sealauren commented 1 year ago

works for producing multipanel plots with msini determined via the api (as long as post.medparams['mpsini'] is set)

bjfultn commented 1 year ago

@sealauren please make sure to pull/merge from next-release and fix any conflicts first.

bjfultn commented 1 year ago

Here is the traceback from the failing Travis tests:

Traceback (most recent call last):
  File "/opt/conda/lib/python3.10/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/code/radvel/radvel/tests/test_api.py", line 139, in test_celerite
    radvel.driver.plots(args)
  File "/code/radvel/radvel/driver.py", line 75, in plots
    GPPlot.plot_multipanel()
  File "/code/radvel/radvel/plot/orbit_plots.py", line 869, in plot_multipanel
    if not notrend:
NameError: name 'notrend' is not defined