add reference genome compatibility modal#161
Conversation
Deploying bedhost-ui with
|
| Latest commit: |
a68595a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4be06fa3.bedhost.pages.dev |
| Branch Preview URL: | https://genome-compatibility.bedhost.pages.dev |
|
Looks good however, we should clarify for SF: Sequence Fit: the proportion of shared region lengths in both this BED file and reference genome over the total number of region lengths in the reference genome [precision].
|
donaldcampbelljr
left a comment
donaldcampbelljr
left a comment
There was a problem hiding this comment.
Awesome. Looks good!
ui/src/pages/bed-splash.tsx
Outdated
| data: genomeStats, | ||
| } = useBedGenomeStats({ | ||
| md5: bedId, | ||
| autoRun: true, |
There was a problem hiding this comment.
If we always want to auto run this you can probably just set it in the query hook
ui/src/queries/useBedGenomeStats.ts
Outdated
| const { data } = await api.get<BedGenomeStatsResponse>(`/bed/${md5}/genome-stats`); | ||
| return data; | ||
| }, | ||
| enabled: enabled, |
There was a problem hiding this comment.
I saw you had autoRun... do we want to always run? Could just get rid of it here
There was a problem hiding this comment.
Updated, just set enabled to !!md5
khoroshevskyi
left a comment
khoroshevskyi
left a comment
There was a problem hiding this comment.
Looks really good. One suggestion, is there a way to add links to refgenie for genomes?
|
I forgot, let me add before we merge. |
4 participants
Added a simple modal with table to show ranking of reference genome compatibilities for now. Is it worth adding the other metrics besides tier_ranking from the /bed/bed_id/genome-stats endpoint?