-
Notifications
You must be signed in to change notification settings - Fork 1
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
API query usability questions and issues #87
Comments
This commit partially connects the viz prototype with the IDM API through a proxy GET endpoint that we host as part of our API. The IDM API cannot be fully integrated with this viz until further design questions are considered. There is a lot of data standardization that needs to be done between IDM and ID3C to avoid lots of if/else or branching logic as part of this app. Currently, the prototype is able to get modeled results for `all` pathogens as well as Flu A H1 & H3. We are getting results for Seattle neighborhoods only. We are not able to query or filter by age, sex, or vaccination status. The time (Week) filter does not currently work in the app, but it should be able to be integrated with React with the current data. 'Mean' is not an available result for every pathogen model. NOTE: The bar chart and the map are not rendering the information correctly. This is still a work in progress that is being put aside until we collaborate further on the future of the modeling results API. Please see [this GitHub issue](seattleflu/incidence-mapper#87) for more context.
This commit partially connects the viz prototype with the IDM API through a proxy GET endpoint that we host as part of our API. The IDM API cannot be fully integrated with this viz until further design questions are considered. There is a lot of data standardization that needs to be done between IDM and ID3C to avoid lots of if/else or branching logic as part of this app. Currently, the prototype is able to get modeled results for `all` pathogens as well as Flu A H1 & H3. We are getting results for Seattle neighborhoods only. We are not able to query or filter by age, sex, or vaccination status. The time (Week) filter does not currently work in the app, but it should be able to be integrated with React with the current data. 'Mean' is not an available result for every pathogen model. NOTE: The bar chart and the map are not rendering the information correctly. This is still a work in progress that is being put aside until we collaborate further on the future of the IDM modeling results API. Please see [this GitHub issue](seattleflu/incidence-mapper#87) for more context.
Thanks for useful questions. Some I can answer now, some need more attention.
4-6) Yes, eventually, depending on how many layers deep we want to go. Right now, I'm just building some subsets because of memory limitations. Addressing this fits "before end of September" timeline for us, but I'll be able to think about it more next week in prep for Ad Board meeting, and so will get back to you with more useful comments then. |
This commit partially connects the viz prototype with the IDM API through a proxy GET endpoint that we host as part of our API. The IDM API cannot be fully integrated with this viz until further design questions are considered. There is a lot of data standardization that needs to be done between IDM and ID3C to avoid lots of if/else or branching logic as part of this app. Currently, the prototype is able to get modeled results for `all` pathogens as well as Flu A H1 & H3. We are getting results for Seattle neighborhoods only. We are not able to query or filter by age, sex, or vaccination status. The time (Week) filter does not currently work in the app, but it should be able to be integrated with React with the current data. 'Mean' is not an available result for every pathogen model. NOTE: The bar chart and the map are not rendering the information correctly. This is still a work in progress that is being put aside until we collaborate further on the future of the IDM modeling results API. Please see [this GitHub issue](seattleflu/incidence-mapper#87) for more context.
Thanks for the quick response, Mike. I did not realize that the targets are coming directly from I don't think there's any rush in expanding the API. Brainstorming API design some time between the Ad Board meeting and the end of September should work for our timeline. I'll reach out if anything changes. |
Re: target vs. organism: we only relatively recently mapped targets to organisms in ID3C. This is important as targets change and vary but detect the same pathogen. It also helps us roll up targets by a less-specific organism classification. So, at some point we should be shipping organism lineage in the modeling observation views instead of targets, and then the models can use them. List of those lineages is here: https://backoffice.seattleflu.org/metabase/question/292 |
Hey all,
Thanks for the work you've done so far in creating/hosting the modelling results API + the swagger interface for us to use.
Now that we're finally coming around to using it, I have a few questions/ideas I thought you may be able to help with.
Examples
I can match your
Flu_A_H1
with our target name pretty easily, but other names are harder to interpret/match with your dataset. For example, I don't see any organism-levelpathogen
for Flu B/Vic or Flu B/Yam, justFlu_B_pan
.Does the
all
pathogen refer to any pathogen? Or does it refer to all samples including negative, untested, and unknown?modeled_intensity_mean
is not a result in every model I found. Ideally we would be able to have mean, mode, sd, and 95% CI for every query.age
outside ofinla_observed
models?For reference, I am currently formatting my queries as:
where
pathogen
is an interchangeable variable I set up on our own, local GET endpoint.The text was updated successfully, but these errors were encountered: