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

Fixes 20 #20

Closed
wants to merge 1 commit into from
Closed

Fixes 20 #20

wants to merge 1 commit into from

Conversation

jvines
Copy link

@jvines jvines commented Apr 11, 2020

Fixes issue 19 that happens when reading old Juliet output with RV instruments and using sigma_w_rv.

The error occurs because the dictionary out['posterior_samples'] is being modified while iterating over out['posterior_samples'].

@jvines jvines changed the title Fixes 19 Fixes 20 May 1, 2020
nespinoza added a commit that referenced this pull request Aug 10, 2020
@nespinoza
Copy link
Owner

Hey @jvines! Just letting you know that I just went ahead and commited this snipped directly (2160e0d) instead of merging this pull request (which has other small changes to the text). Thanks for this!

@nespinoza nespinoza closed this Aug 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants