Skip to content

Conversation

@jjaassoonn
Copy link
Contributor

This PR includes the problem statements from Putnam 2025

@jjaassoonn
Copy link
Contributor Author

Now let me address the CI issues.

@jjaassoonn
Copy link
Contributor Author

Seems the errors are all because of missing linebreaks. Hopefully it is okay now.

Let $S$ be the sum of the entries of $A$, and let $N$ be the number of nonzero entries of $A$.
Prove that $S \leq \frac{(n+2)N}{3}$.
Note: We parameterize by $m$ where $n = m + 2$, so $m \geq 0$ and the matrix is $(m+2) \times (m+2)$.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this will fail CI because the Note isn't part of the informal JSON (CI checks docstring is the same). Perhaps you can move this note elsewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry missed this, one. Just pushed a fix.

@jjaassoonn
Copy link
Contributor Author

Working on a fix right now.

@jjaassoonn
Copy link
Contributor Author

By the way is there a way to automatically extract all the doc strings?

@GeorgeTsoukalas
Copy link
Collaborator

By the way is there a way to automatically extract all the doc strings?

I don't think we have code checking in anywhere for that unfortunately. A simple script should be able to handle this I think

@jjaassoonn
Copy link
Contributor Author

Just added the missing newline character in b1,b2,b3

@jjaassoonn
Copy link
Contributor Author

Is the current CI failure possibly caused by some definitions are not part of the problem statement?

@GeorgeTsoukalas
Copy link
Collaborator

Thank you so much for the contribution, Jujian!

@GeorgeTsoukalas GeorgeTsoukalas merged commit 87fcd63 into trishullab:main Dec 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants