-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update load_kegg_KAAS.pl to fit to new format of http://rest.kegg.jp/list/genome #7
Comments
Hello there, I just would like to check whether fixing this bug is on the horizon. Do you have an estimation on when it will be addressed? Sorry to bug. I really liked the pipeline. It is comprehensive and does what it proposes very well. Fa-ng became essential for some urgent important projects of mine. And that is why I am asking. Cheers, |
Hi @LucasMS , thanks for reporting. I just noticed your issue. Let me take a look and I will follow the conversation here...
|
Hi @toniher, Thanks for coming back. I started digging a bit on the issue. I could not find a version online of the KEGG genome list with the taxid info. Actually, I did not find any good resource on the KEGG non-subscription files. So, I found a work around, which I think is not the best solution. I am trying to match the KEGG genome list to the Taonomy This approach is not straight forward, though. Names are not fully compatible between the two resources. A simple example is Anyway, I needed to move on with my projects :). I implemented this matching with an R script (I am not familiar with perl) and I am currently trying to implement it into the pipeline. I think this dirty fix should work for me. But will be great to have a proper solution for it :) I hope this is somehow helpful for you. |
Hi @toniher, I am still trying to fix this somehow on my side. I want to check if my modifications in the load_kegg_KASS.pl script have worked. But I can not find the perl packages FunctionalAnnotation::DB and unctionalAnnotation::uploadData. Do you know how can I get/install those? Thanks! |
Hi @LucasMS these packages are at |
Thanks, @toniher . I find a work around for the load_kegg_KASS.pl. I used the strategy that metioned I mentioned above, where I combine combine genome list and NCBI taxonomy and load the formatted file into load_kegg_KASS.pl. I am not sure, if it is working as intended, though. Now I get an error down the pipeline:
I actually not sure what is going on. And I am afraid I can not fix this alone. Cheers, |
Hi @LucasMS , share the changes you made to the file. This weekend I will try to find some time to address your issue and include other updates if possible... |
Hi,
Thanks for the pipeline!!
I think that KEGG annotation results are not being properly loaded in the database. I observed that the record at the rest.kegg page is no longer
but now
I do not think that the script is parsing it correctly anymore and no taxonomy Id is provided by KEGG.
Cheers,
Lucas
The text was updated successfully, but these errors were encountered: