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

[feature] exclude some deps from conan dependency graph #9303

Closed
boussaffawalid opened this issue Jul 21, 2021 · 7 comments · Fixed by #16083
Closed

[feature] exclude some deps from conan dependency graph #9303

boussaffawalid opened this issue Jul 21, 2021 · 7 comments · Fixed by #16083
Milestone

Comments

@boussaffawalid
Copy link

Conan dependency graph is a very useful feature but it become very difficult to use when having too many dependencies.

This is how our current dependency graph looks like, it is almost unreadable and we are adding more dependencies.

Screen Shot 2021-07-21 at 2 30 38 PM

It would be very handy to add the ability to exclude some dependencies from this graph. For example excluding third party deps that rarely get changed.

@memsharded
Copy link
Member

Hi @boussaffawalid

I think the best approach for this at the moment would be to customize the html/javascript template that is used to generate this graph, not to do it from the Conan client side: https://docs.conan.io/en/latest/extending/template_system/info_graph.html

I understand this would be a nice to have, but at the moment this wouldn't be a priority. If someone is willing to contribute an improved html/js template for the graph, with more powerful features, like hiding/showing packages, etc., it will be very welcome.

@boussaffawalid
Copy link
Author

@memsharded it seems that info_graph.html does the trick! Thanks!. I will try to contribute to the html/js template.

@memsharded memsharded added this to the 2.X milestone Feb 14, 2024
@Bbox123
Copy link
Contributor

Bbox123 commented Apr 4, 2024

Hello, me and my UMICH group want to contribute to this issue! I assume this is still available since it has the appropriate milestone mark. Do you happen to have any suggestions on how to start tackling this issue? Thank you!

@memsharded
Copy link
Member

The html graph got a very major update very recently in #15846

I think that PR would alleviate most of the problems referenced in this ticket., but the "filtering" one is not.
I suggest adding another textbox that accepts a comma separated list of patters of packages to hide.
If we want to make the functionality equivalent in the search box, it could be done the same, and extend the search to a comma separated list of patterns.

@Bbox123
Copy link
Contributor

Bbox123 commented Apr 4, 2024

The html graph got a very major update very recently in #15846

I think that PR would alleviate most of the problems referenced in this ticket., but the "filtering" one is not. I suggest adding another textbox that accepts a comma separated list of patters of packages to hide. If we want to make the functionality equivalent in the search box, it could be done the same, and extend the search to a comma separated list of patterns.

Thank you! We will move forward with that implementation in mind.

@Bbox123
Copy link
Contributor

Bbox123 commented Apr 12, 2024

Hello again, our group has implemented a solution based on your suggested design and are hoping to open a pull request by this upcoming week. We just had a question about the contribution guidelines. The guideline step 7 is to make a PR to the conan docs. Should we only move onto this step if our implementation PR is approved? Thanks!

@memsharded
Copy link
Member

Thanks! You can skip the PR to the docs, the html page should be self explanatory, the box should contain a default "Filter packages" or something like that, can have a hover help too.

Just put Docs: Omit in the PR.

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

Successfully merging a pull request may close this issue.

3 participants