Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chopping chains in results class #193

Open
semaphoreP opened this issue Nov 17, 2020 · 4 comments
Open

chopping chains in results class #193

semaphoreP opened this issue Nov 17, 2020 · 4 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@semaphoreP
Copy link
Collaborator

I don't think we currently have the ability to read in posteriors from MCMC that are saved as a HDF5 file, and chop chains in the results class. This can be useful when we identify we have further burn-in that needs to be removed but don't want to rerun the whole MCMC.

@semaphoreP semaphoreP added the enhancement New feature or request label Nov 17, 2020
@sblunt sblunt added this to the v2.0 milestone Apr 21, 2021
@tomasstolker
Copy link
Contributor

Great tool! I just did my first orbit fit :).

I have a question a bit related to the suggested enhancement by @semaphoreP.

What is the correct way to reshape the post array from Results backward to a shape in which the walkers have their own dimension?

I am guessing reshape(n_walkers, n_steps, 8) but I wasn't completely sure. Thanks!

@sblunt
Copy link
Owner

sblunt commented May 25, 2021

Awesome, thanks for using orbitize! @tomasstolker

Yup, what you suggested should work. Just remember that the total_orbits input to MCMC.run_sampler is actually num_steps_per_walker x num_walkers, so in your reshape command, n_steps should be the total_orbits input divided by num_walkers.

@sblunt sblunt removed this from the v2.0 milestone Aug 25, 2021
@sblunt sblunt added the good first issue Good for newcomers label Nov 2, 2021
@semaphoreP
Copy link
Collaborator Author

This should be higher priority!

@semaphoreP
Copy link
Collaborator Author

This feature is in @manduhmia's plot_all_orbits branch. We should merge that branch in soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants