-
Notifications
You must be signed in to change notification settings - Fork 120
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
Welsh powell coloring #391
Conversation
Hi @badumbatish , please resolve conflict so we have the possibility to merge it! |
Hi there, I forgot to sync my fork with the origin repository. Things should be fixed now @ZigRazor |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #391 +/- ##
==========================================
+ Coverage 97.55% 97.58% +0.03%
==========================================
Files 85 87 +2
Lines 9367 9494 +127
==========================================
+ Hits 9138 9265 +127
Misses 229 229
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Hi there, I'm creating a pull request to merge the Welsh Powell Coloring algorithm into the master branch.
The algorithm returns a std::map<Node, int> result that assigns each node to a color order (assigned by integers).
Users can also query the minimum chromatic order of the graph by querying the highest value from the resulting map
The minimum coloring starts from 1 instead of 0.
The algorithm assumes the graph to be undirected. All sources and inspirations are linked in the declaration of the algorithm and test cases.