Skip to content
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

Serratia MLST Schemes #136

Open
erinyoung opened this issue Mar 21, 2024 · 3 comments
Open

Serratia MLST Schemes #136

erinyoung opened this issue Mar 21, 2024 · 3 comments

Comments

@erinyoung
Copy link

I learned through the ubioinfo slack channel that there is a MLST scheme for Serratia, but it doesn't appear to be included in the files that mlst-download_pub_mlst downloads.

Is there a reason this organism hasn't been added? Is there a way I can request that this organism gets added?

@schorlton-bugseq
Copy link

Hi @erinyoung, you may have figured this out by now but if not, this issue is not specific to this tool. The download script pulls the XML from PubMLST (https://github.com/tseemann/mlst/blob/v2.23.0/scripts/mlst-download_pub_mlst#L11), and the PubMLST XML file is missing the Serratia scheme. The XML or download script can be modified to add the missing scheme. Hope that helps!

@erinyoung
Copy link
Author

Is there a recommended way to edit the xml file or download script?

@schorlton-bugseq
Copy link

Not sure "recommended" but you'll notice the XML follows a pretty standard format so the serratia scheme with its loci can be slotted in and then the download script hacked to read from local XML or the Serratia files can be manually downloaded and placed in the folder with the rest of the schemes. Probably best/easiest to report this to PubMLST though? If they fix it (I would hope it would be a trivial fix on their end?), then no modifications would be needed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants