Skip to content

Remove vg msga#4943

Merged
adamnovak merged 23 commits into
masterfrom
rm-msga
Jul 17, 2026
Merged

Remove vg msga#4943
adamnovak merged 23 commits into
masterfrom
rm-msga

Conversation

@faithokamoto

Copy link
Copy Markdown
Contributor

Changelog Entry

To be copied to the draft changelog by merger:

  • vg msga is removed

Description

Messed with a lot of test stuff, but it doesn't rely in vg msga any more. You know what does rely on it, though? The wiki.

I'm not entirely sure what to do about those buggers. I guess for the first two I could just run the command and save the output? But the last one is that sort of broken page that I would now be breaking even further.

@adamnovak since you asked for it :)

@faithokamoto

Copy link
Copy Markdown
Contributor Author

It failed some doc job /builds/vgteam/vg/doc/../README.md that I have no idea where it is. Looks like we'd just need to delete some line that tests msga.

@faithokamoto

Copy link
Copy Markdown
Contributor Author

Oh, silly. That means the repo base level README.

@adamnovak

Copy link
Copy Markdown
Member

For the last one (https://github.com/vgteam/vg/wiki/VG-RDF%2C-the-Ensembl-bacteria-E.-coli-genome-hack-attack) we could try vg map or vg giraffe, if they work on such long inputs, to get a GAM, and then vg augment to add the GAM to the graph.

@faithokamoto

Copy link
Copy Markdown
Contributor Author

I think the last one is begging for me to actually try to rerun the entire example. I'll put it back on the wiki update list...

@faithokamoto

Copy link
Copy Markdown
Contributor Author

Reworked the tutorial to not use vg msga: https://github.com/vgteam/vg/wiki/Graph-&-annotations-for-two-E.-coli-strains

Will plug it into everywhere relevant tomorrow, I'm about to go watch Project Hail Mary again :)

@faithokamoto

Copy link
Copy Markdown
Contributor Author

@adamnovak anything else we might need to worry about before this gets merged in?

@adamnovak

Copy link
Copy Markdown
Member

I think that's probably it.

@adamnovak
adamnovak merged commit 6caf2ef into master Jul 17, 2026
2 checks passed
@faithokamoto
faithokamoto deleted the rm-msga branch July 17, 2026 21:49
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

Successfully merging this pull request may close these issues.

2 participants