search

j-faria / kima

Exoplanet detection in RVs with DNest4 and GPs
http://www.kima.science
MIT License
15 stars 10 forks source link

Memory usage #15

Closed eggplantbren closed 6 years ago

eggplantbren commented 6 years ago

I was running kima on a dataset of mine and it used up all my RAM and started using virtual memory. Obviously this is a problem that needs to be fixed.

j-faria commented 6 years ago

Is this related to #13?

eggplantbren commented 6 years ago

I don't know yet...

On 5/12/2017 1:32 PM, "João Faria" notifications@github.com wrote:

Is this related to #13 https://github.com/j-faria/kima/issues/13?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/j-faria/kima/issues/15#issuecomment-349154756, or mute the thread https://github.com/notifications/unsubscribe-auth/ABgVOgS1AR21ikHh9LkSFrS5rYEysRxbks5s9I8qgaJpZM4Q1iq7 .

eggplantbren commented 6 years ago

Resolved (inelegantly) with b8341622bc9d05b2be2ea89a635438b50059efe6. This might help with #13.

j-faria commented 6 years ago

I was thinking this could be solved with a setpars method in each ContinuousDistribution such that

Pprior->setpars(center, width);

would work (assuming Pprior is Laplace)