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

Request software updates #15

Closed
Xuelei-Dai opened this issue Dec 8, 2023 · 6 comments
Closed

Request software updates #15

Xuelei-Dai opened this issue Dec 8, 2023 · 6 comments

Comments

@Xuelei-Dai
Copy link

Hello Daniel,

I found that the version of the panacus is no longer compatible with the results of Mingraph-Cactus (MC), and the final visualized graph is incorrect. I think this problem needs to be completely solved. It is necessary to use the current Mingraph-Cactus pipeline to construct the pan-genome and use the final results for visualization.

Best wishes
Xuelei

@danydoerr
Copy link
Member

Thanks for reporting! Can you provide a graph where I can debug the issue?

@glennhickey
Copy link

I don't have any ideas of mc changes that would affect Panacus. Here's two yeast graphs made with Cactus v2.6.4 (which was used for the v1.1 HPRC graphs) and Cactus v2.7.0 (which is the current release).

If there's a regression, I would think it would show up between these two graphs.

http://public.gi.ucsc.edu/~hickey/debug/panacus/

@danydoerr
Copy link
Member

Thanks @glennhickey!

@danydoerr
Copy link
Member

@Xuelei-Dai
Copy link
Author

Thanks @danydoerr and @glennhickey!

In the current version of Mingraph-Cactus (MC), the gfa file does not have P line, and the new W line was changed. Since I referred to the Pangenome Coverage and Growth Statistics pipeline (convert the 1.1 version of gfa to 1.0 (vg convert -gfW)), recently I directly used the 1.1 version of the gfa file as the input of panacus, it's okay. So I suggest that you can add some instructions to your pipeline, for example, how should the gfa 1.1 version be run?

Best,
Xuelei

@danydoerr
Copy link
Member

That's a good point, @Xuelei-Dai, the current README does not contain an example for the MC graph. I have now added an example for the MC graph.

To be perfectly clear, panacus has always supported GFA 1.1 and therefore also the latest MC graphs. Neither any update is required, nor any bug to be fixed. The example in the README suggests to visualize pangenome growth only for a subset of samples (those corresponding to haplotypes). This preprocessing is not part of panacus and remains the responsibility of the users.

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

3 participants