dsavransky / EXOSIMS

Simulator for exoplanet direct imaging space missions
BSD 3-Clause "New" or "Revised" License
25 stars 35 forks source link

Planet mass should be generated in gen_plan_params #315

Open dsavransky opened 1 year ago

dsavransky commented 1 year ago

Is your feature request related to a problem? Please describe. Currently, in PlanetPopulation modules, planet paramters a, e, p, and Rp are generated by calling gen_plan_params, while planet mass is generated by calling gen_mass. This causes issues and requires silly workarounds for populations (i.e., DulzPlavchan) that require planet mass to be based on radius. More importantly, we should always allows for correlation between any or all of these parameters.

Describe the solution you'd like Planet mass should be added as an output to gen_plan_params, and gen_mass should be eliminated as a separate class method.

Additional context These are the modules defining/calling gen_mass:

Here are the modules defining/calling gen_plan_params: