-
Notifications
You must be signed in to change notification settings - Fork 17
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
WIP: Cool Results Page #63
Conversation
…re off ajax requests
Thanks, @dhakim87! This is looking pretty good. I had some follow up with our comms team, and they were suggesting opening with some fairly high level text like the stuff below. Some of which may require some additional bits on the public API (cc @gwarmstrong), like the rare/unique bit, which is part of what I'm trying to knock out this week. I'll email the mockups of the other pages, which are at the moment mostly just the content from the current results page with the exception of taxa ranks and some more user-guided bits on the existing alpha diversity plot. @gwarmstrong, with the nearest neighbors piece, I think right now we could do some basic text stuff like this (i.e., the person you're most similar to is XXX years old, and eats sugary sweets with YYY frequency) but I think under the diversity similarities section. I'll summarize a bit more on slack today. It would be advantageous for the nearest neighbors bits to be dataset aware, so that we could (for example) depict what mammal someone is most similar too.
|
|
||
init = function(){ | ||
// TODO: Should data type options be determined by ajax call or | ||
// passed into jinja2 template? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably ajax as this is coming from the public api. If a sample was only subject to one data type, then the pulldown shouldn't be selected
"16s" | ||
); | ||
|
||
// TODO: Should dataset options be determined by ajax call or |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably ajax as this information is coming from the public api
AFAIK the
This seems like a slightly different problem than the neighbors endpoint supports. The For the mammals bit, it seems we may want an additional feature that supports "get the k most similar samples in this dataset that meet a certain criteria", which is different because it would also involve searching for samples that meet the criteria. |
Correct, although it doesn't need to be expressed as a search but instead a filter prior to sorting. We need to expand slightly to support overall nearest neighbors, and a restricted nearest neighbors |
Moved to new issue: biocore/microsetta-public-api#74 |
…xisting results page implementation
…re URLs for bacterial and archaeal counts
…ng while we finish up public api server
…:none to display:block. We now trigger a resize event on the window when the emperor tab is selected, which emperor links to determine canvas sizing in openGL.
… Fixed missing column headers on taxonomy table
Emperor width workaround
Results Page: Taxonomy Table
…unds for jquery event handling while Emperor is loading in
…ecies, summing their relative abundances.
WIP: How you compare tab of cool results
…t on routes that didn't specify dataset
… then prevented emperor from being reloaded later
…ould also show a tooltip saying why, but waiting on popper inclusion in another branch
…rce (that has no samples associated with it)
…e info for environmental or animal samples
Work In Progress, don't merge this yet!
Skeleton for new results page with events linked, showing where to fire off ajax requests and what to call to update controls. I need to know what words to put on each of the four tabs, what kind of plot(s) to show in that tab, and what urls to hit on public api to grab the data for that tab.