Skip to content

Conversation

@Robertboy18
Copy link

Pushed #339 and also related problem https://www.erdosproblems.com/740 which is basically the infinite version in some sense to 108!

@google-cla
Copy link

google-cla bot commented Jul 27, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Member

@Paul-Lez Paul-Lez left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! A few minor nitpicks


*Reference:* [Erdős Problems](https://erdosproblems.com/108)

For every r≥4 and k≥2, does there exist finite f(k,r) such that every graph of chromatic number ≥f(k,r) contains a subgraph of girth ≥r and chromatic number ≥k?
Copy link
Member

Choose a reason for hiding this comment

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

The convention we're following is to have the statement of the theorem as a docstring above the result rather than in the module docstring.

@Paul-Lez Paul-Lez added the Awaiting author A reviewer has asked the author a question or requested changes. label Aug 18, 2025
@mo271
Copy link
Collaborator

mo271 commented Sep 5, 2025

@Robertboy18 would you be able to sign the CLA or do you need help, for instance changing the email address used in the pull request if another one is already associated with signing the CLA?
View this failed invocation of the CLA check for more information.

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

Labels

Awaiting author A reviewer has asked the author a question or requested changes. Erdős Problems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants