-
Notifications
You must be signed in to change notification settings - Fork 0
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
Missing Data: mixed.qtl.Hwang_King_2016 #206
Comments
@jd-campbell not %100 sure but this sounds potentially related to some other issues that I'm guessing may stem from scripts that Sam had written to generate the files for soybean QTLs from the info in the soybase mysql. At present I have no clue as to where those scripts may be but will send a flare up to Sam and see if he has any recollection of where he might have put them. |
Sam was super-fast and helpful in his response. The scripts are here: |
@adf-ncgr Thanks for the info. This helps in my work. Please send my thanks to Sam also! |
@jd-campbell not sure this one is quite ready to be closed, but here's an update. I got Sam's code to run and for this dataset it seems to have produced 26 QTLs, although one of them (mqCanopy wilt-013) looks like it may be problematic without location info:
In any case, I'm not sure why the datastore file would only have 2 QTLs since this one seems more complete (though maybe still not entirely complete?). I'll try to explore a little more but wanted to let you know there's at least some progress on this. |
OK, it looks like the issue with that one QTL without location info is probably a data issue, and not the fault of the code. mqCanopy wilt-013 is one of ~40 QTLs without an entry in the qtl_position_table :
Also note that the db seems to have only 26 not 27 QTLs (at least, per select count(*) from qtl_table where QTLName like 'mqCanopy wilt%'), so I think the version I got out of running the code is probably close to correct. Let me know if you think that one QTL missing a position can be fixed in the db, otherwise I'll just replace the datastore file with the new one. |
@adf-ncgr @jd-campbell Since the paper says that mqCanopy wilt-013 (QTL name 5-2) is only associated with Satt229, the position values should be 92.88 94.88 93.88. That is 1 cM on each side of Satt229 which the database says is at 93.88 on LG L or Gm19. I am not sure why it was left out, but this record was problematic and had to be adjusted after the data was originally entered by an undergrad student worker. I have inserted mqCanopy wilt-013 into both stage and production MySQL databases. |
I have noticed that there is missing data in the mixed.qtl.Hwang_King_2016 directory. The *qtl.tsv file only contains 2 QTLs but the SoyBase MySQL database lists 27 QTLs.
@jd-campbell Will review the paper and SoyBase MySQL to ensure all the data is in the DS.
The text was updated successfully, but these errors were encountered: