-
Notifications
You must be signed in to change notification settings - Fork 13
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
Working Group Use Case #3 Extensions #394
Comments
From : 20240731 Summary of Meeting 31 July 2024
|
part of #394 manage-genotype.js : vcfGenotypeReceiveResult() : for genotype-search, i.e. ! scope, use showFeature() for added.createdFeatures, which are all features from the result. package.json : update vcf-genotype-brapi dependency to 1.0.3, which includes changes for WG#3, supporting genotype-search.
Extension 1: Show triangles for features foundThis was implemented via source code changes in frontend GUI, which were able to leverage existing library function transient.showFeature(). Changes are complete and tested before commit. The commit is included in images that are built for the dev servers.
|
Additional upgrade measuresDuring testing, some exceptions were encountered from undefined values in 'Genotype Search' resulting from the merge with the upgraded version. pretzel plantinformatics/vcf-genotype-brapi |
Display of Feature Triangles beside axis and display of SNPs in Genotype Table get intermittent results in user testingUsers exercising the 'Genotype Search' functionality on our internal test server reported that in some cases it only draws one triangle and isn't displaying all the SNPs in the table. Also, one of the SNPs requested was not displayed in the Genotype Table. In one instance this was because of a typographical error in the SNP Name (missing suffix '-F'). Retest with the given test case used (Rht-B1,Rht-D1,Pina-D1,Pinb-D1) to check these 2 issues. ExtensionsUsers suggested the SNPs in the Genotype Table could be ordered by chromosome, then position; at the moment it orders by position only. InvestigationThe data which demonstrated the issue is on the dev server, so the cause was identified by using local + dev server to have the benefit of un-minified source code when debugging; the process would have taken >5x longer to find in the (minified) production build, i.e. without multiple servers. SolutionThe locations of the found SNPs are used to set the domain of the brush, and a calculation which used scale to convert back from brush pixels to brushed domain resulted in a loss of precision :
The solution implemented is to use the feature domain directly (exact) when possible, as in this case. 9331e9d ensure brushedDomain is exact so that outside SNPs are not filtered out of table |
Part of #394 Factor from manage-genotype.hbs to form genotype-samples.hbs, changing this. to @the. manage-genotype.hbs : use <Panel::GenotypeSamples > in place of factored code. add genotype-samples.js, with actions vcfGenotypeSamples(), nameFilterChanged(). genotype-search.hbs : use <Panel::GenotypeSamples>. Add class=genotypeSearch for width of textarea, select. genotype-search.js : change selectedSamplesText to alias to manageGenotype.selectedSamplesText. vcfGenotypeSearchDisabled() : add .length to dependencies, seems needed for update when selectedSamplesText changes. add manageGenotypeDefined, set in navigateGenotypeTable(). navigateGenotypeTable() : wrap manageGenotype.vcfGenotypeSamplesSelected assignment in later() to avoid : update ... used previously in the same computation. app.scss : .genotypeSearch select.form-control, textarea : width: 100%. left-panel.hbs : pass userSettings to <Panel::GenotypeSearch > genotype-samples-test.js : added by ember g component panel/genotype-samples, not edited.
Extension 2 : sample (search) filter (factor from Genotype Table controls)To enable users to identify samples, the functionality of sample name filtering which is available in the Genotype Table settings / controls dialog is utilised also in the 'Genotype Search' panel. The design change is to factor the functionality into a component genotypes-samples, which is then used in both the Genotype Table settings / controls dialog and the 'Genotype Search' panel (i.e. manage-genotype and genotypes-search components). This commit e1aee89 contains most of the changes required; the remaining work is to connect updates from dataset selection so that samples are requested and the display is updated.
This commit is a follow-on from the work in #370.
These commits add request and display of samples from user change to selected dataset.
This screen shot shows the status. Remaining work is to not show these visual elements :
Those last 2 points are now implemented by these commits : |
Extension 3 : Order the Genotype Table first on chromosome, then positionCurrently the Genotype Table displays rows in position order.
|
Fix reported issue : in this test, only one of the triangles is displayed101c3b2 ensure manageGenotype is shown before setting selectedSamplesText |
Completed and ReleasedThis work is complete and included in the v3.0.0 release #414 , and has been tested in production in that release. |
(part of #383)
Introduction
Testing of the prototype showed that it met the MVP requirements of #383.
Testing also suggested a number of usability enhancements which could be made to improve user ergonomics and suit the expected user work-flow better. These are noted in the following, and will be investigated and where the effort is reasonable for the benefit, these will be completed.
Suggested Enhancements
it is desirable to enable the user to append new names to either : samples / features, and issue a repeated request, and see the results
Add a button 'Clear Genotypes' to enable the user to start afresh after accumulating results from 1 or more searches.
Preferably the axes should be sorted in numeric/alpha order from left to right when first displayed (the user can rearrange these).
Disable the search button during the request
After request, indicate if markers are not found, e.g.
Set View : Outside Axis Margin to 0, to make more room for the Genotype Table, which is the primary display rather than the axes.
The text was updated successfully, but these errors were encountered: