BDNYC / sedkit

Spectral energy distribution construction and analysis tools
MIT License
16 stars 13 forks source link

Same Radius #77

Closed graceparkk closed 1 year ago

graceparkk commented 2 years ago

HI! I've been running various objects with the SEDkit and found out that it gives out the same radius for all objects; regardless of the object, it gives out 0.9731158731046832 jupiterRad.

hover2pi commented 2 years ago

Ok I'll take a look! Thanks for letting me know.

SherelynA commented 2 years ago

Hello! @hover2pi I just wanted to add more info to this error that we are getting. The line of code we get when initializing the sed in particular for the radius says Setting radius to (<Quantity 0.97311587 jupiterRad>, <Quantity 0. jupiterRad>, <Quantity 0. jupiterRad>) with reference '2013ApJS..208....9P. Overall, is the radius of the objects being inferred using a specific method?

hover2pi commented 2 years ago

Ok, the fix for this is in the next release #80

SherelynA commented 2 years ago

thanks, super excited for the new release!