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

Obs priors #365

Merged
merged 46 commits into from
Apr 11, 2024
Merged

Obs priors #365

merged 46 commits into from
Apr 11, 2024

Conversation

sblunt
Copy link
Owner

@sblunt sblunt commented Mar 1, 2024

Implementation of the observation-based priors (finally!)

I added a new basis to match the one derived in Kelly's original paper, and modified the code on my old branch to implement the math correctly. @clarissardoo and I also wrote up a tutorial, and I added some unit tests. I'm running a longer end-to-end fit (using the new tutorial, just running for a goodly number of steps), and I'll post the results from that here so we can visually compare to a published independent implementation.

@sblunt sblunt requested a review from semaphoreP March 1, 2024 23:37
@sblunt sblunt changed the base branch from main to v3 March 1, 2024 23:37
@semaphoreP semaphoreP self-assigned this Mar 28, 2024
Copy link
Collaborator

@semaphoreP semaphoreP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix image in docs. and we should be good

@sblunt
Copy link
Owner Author

sblunt commented Apr 11, 2024

Image is displaying for me in the tutorial, and the CI run through the notebooks passed. Not sure why you couldn't see it. I'm going to pull this in but we can fix it later if it's still not working.

@sblunt sblunt merged commit 0412dc0 into v3 Apr 11, 2024
8 checks passed
@sblunt sblunt deleted the obs-priors branch April 11, 2024 22:43
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

3 participants