-
Notifications
You must be signed in to change notification settings - Fork 36
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
store pca scores results #5265
base: master
Are you sure you want to change the base?
store pca scores results #5265
Conversation
…one pca page work
…JAX/Search/Trait.t test
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.
:-)
I ran the db patches and tried to save PCA results and was met with an error... I sent a slack message about it |
Please run this: #5038 and follow this message from @naama: you will probably find 2, and you need to have only one with a cv_id of the 'relationship ontology' you want to delete the duplicated 'variable_of 'cvterm , but before deleting it you want to update your cvterm_relationship table and do an update begin; update cvterm_relationship set type_id = CORRECT_VARIABLE_OF_CVTERM_ID where type_id = CVTERM_ID_TO_BE_DELETED (edited) |
Tried this, still does not work. Current issues I get (running breedbase docker):
I suspect all of these issues are caused by the cvterm problem, but I do not know how to fix it |
The broken analysis detail page is fixed #5264. I merged master (with the fix) to the branch, so that should be resolved. The 'variable_of' cvterm with the reference to the right cv should be in the database. That is required for anything related to traits. Could you reproduce how the results links are all showing at the bottom, instead of right below the corresponding plots? |
|
Thanks, Ryan. I reproduced it. If you followed the results link from your profile page, it should have loaded the results in the pca page like below. Doesn't it do the same for you? It is not necessary to click the run pca button again for the same dataset, otherwise the same output is displayed again. I will think about a solution for it anyway. |
Yes it works so long as one does not try to run the pca again for the same dataset. However, I am still getting an error with actually saving the results. When clicking the save PCs button, I get the following error message: Any idea why the plots would be invalid? It could be related to the data in the test fixture - not sure. |
…r clicks run pca btn
fixed 7e74097 |
Description
-- adds an option to store the score of top 10 principal components. For now, the option is available if a user submitted the pca job analysis to run in the background and expected an email notification about the job status.
-- adds pca stat terms to the cxgn_statistics.obo file
-- updates pca selenium tests
-- updates fixture with the db patches and loaded cxgn_statistics.obo
-- updates mech tests in t/unit_mech/AJAX/Search/Trait.t, t/unit_mech/AJAX/_BrAPI_v1.t, t/unit_mech/AJAX/_BrAPIv2_phenotyping.t. Updates expected counts in records, pages, total cvtermes, etc., to reflect new cvterm entries in the fixture db.
Required:
-- run db patch
-- reload cxgn_ontology.obo
-- potentially delete double 'VARIABLE_OF' cvterm entry in the database
Checklist
/t/data/fixture/patches/
./docs
has been updated./js
to/js/source/legacy
.