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

US likelihood #207

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

US likelihood #207

wants to merge 4 commits into from

Conversation

jcblemai
Copy link
Collaborator

added option for aggregation of sims and obs to total level (i.e., fo…r US overall) and inclusion in likelihood. Also included option to multiply the likelihood of the total by a weight to provide it with more pressure on overall fits.

shauntruelove and others added 2 commits April 10, 2024 16:48
…r US overall) and inclusion in likelihood. Also included option to multiply the likelihood of the total by a weight to provide it with more pressure on overall fits.
@shauntruelove
Copy link
Member

@jcblemai is this one ok to merge? seems you created a separate PR, not sure which is correct.

@jcblemai
Copy link
Collaborator Author

I created two PR because there were two bases branch, and I don't know what is the difference.

  • breaking-improvements_USlikelihood (this one)
  • inference_with_usa (Inference with usa #206, this one has conflict with main).

so I wanted to have them visible (otherwise branch are forgotten when there is no PR open). Now @shauntruelove I cannot approve them but once these are reviewed (and conflict solved for the other one) they can be merged.

@jcblemai
Copy link
Collaborator Author

Requested a review from alison and you here so we can merge this one at least

Copy link
Member

@shauntruelove shauntruelove left a comment

Choose a reason for hiding this comment

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

looks good

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