-
Notifications
You must be signed in to change notification settings - Fork 345
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
[MER][T11|T12|T13|T14] Creating the initial problems graph #1891
[MER][T11|T12|T13|T14] Creating the initial problems graph #1891
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @syslaila,
Thank you for this PR! I think we can remove some of the classes to make it more transparent.
Happy to send a PR, but I would need your input on some of the questions.
Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_util.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_util.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_util.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/src/core/problems_graph_util.cpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_util.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/src/core/problems_graph_util.cpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_creator.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/src/core/problems_graph_util.cpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com> Update libmamba/include/mamba/core/problems_graph_util.hpp Co-authored-by: Antoine Prouvost <AntoinePrv@users.noreply.github.com>
5d1521a
to
39de2b4
Compare
I seems like the last failing test is due to a temporary connection error
|
Awesome stuff. It all looks good to me (except for two changes that could be their own PR: in util_random.cpp and in subdirdata.cpp – both don't seem related to the core of this PR :) The other remark I had was that it could be nice to add more comments in the |
Few more fix from reviewers feedback
Graph creation implementation for #1878 (we only implemented the graph creation for a subset of the libsolv problems)
We create the graph using the libsolv problems list. We will only add the nodes in the graph if they haven't been added already. In order to check if a specific package is in the graph or not, we store a mapping between the SolveId (a unique identifier from libsolv) and Id (the unique node identifier for a node in the graph).
Because a solvable can appear in multiple libsolv problems, we might need to update the information for the node.
eg.