-
Notifications
You must be signed in to change notification settings - Fork 2
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
handle sample types in linear_pool
#27
Comments
More detailed ideas about handling samples. See comments here and below where we have talked about this before. Previous decision: This function will not do any validations related to potentially different dependence structures represented by the component models. If a hub cares about enforcing that models use the same dependence structure for their samples, this will be specified in the hub's config, and will be checked at the time that model outputs are submitted to the hub, so we don't need to do validations related to this in this function. There are two cases. Case 1: collect component samplesIf all three of the conditions in points below are satisfied, this function simply collects the samples from the component models and updates the sample indices to ensure that they are different for different component models:
Case 2: do some samplingHowever, if any of those conditions are not satisfied, we have to do something else:
Notes about a new function argument related to desired ensemble sample sizeThe sampling step in case 2 requires the user to specify how many ensemble samples they want in the output. So we need to add an argument to We can have 3 separate validations related to this:
|
Couple of additional thoughts about this:
|
If we do something like this, maybe we should introduce an argument like |
I have re-opened this issue as it would provide features that would be useful to our group at UMass, both for our internal modeling and for administering the variant nowcast hub. |
We pretty much settled on the desired functionality here in discussion on issue #20, but I'm splitting implementation into a separate issue.
The text was updated successfully, but these errors were encountered: