Skip to content

New addition: Chromatic numbers! #18

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

Merged
merged 5 commits into from
May 22, 2025
Merged

New addition: Chromatic numbers! #18

merged 5 commits into from
May 22, 2025

Conversation

joshuaguerin
Copy link
Owner

Updates:

  • Added a modified version of the chromatic numbers implementation by @gonz9032. Now doesn't need an input parameter, and added some count macros to clean up the presentation a bit.
  • Updated the README to include information about both versions (k-colorability and chromatic numbers).
  • Updated the primary README to add chromatic numbers as a table entry, linking to the colorability repo.
  • Fixed a minor typo in the table where "First Header" and "Second Header" were still there from templating the table.

@joshuaguerin joshuaguerin requested review from gonz9032 and mads1158 May 19, 2025 22:05
@joshuaguerin joshuaguerin added the enhancement New feature or request label May 19, 2025
@joshuaguerin
Copy link
Owner Author

Still not certain what sort of merging we will be using. I'm leaning towards either a squash or rebase.

Copy link
Collaborator

@gonz9032 gonz9032 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks good to me!

@joshuaguerin
Copy link
Owner Author

OK, I think things are good enough for inclusion for the moment. I'll merge, and we can do a more thorough review of everything later.

@joshuaguerin joshuaguerin merged commit 356685a into master May 22, 2025
@joshuaguerin joshuaguerin deleted the guerin-a branch May 22, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants