Support multi-line PARI formulas for multiple functions#582
Support multi-line PARI formulas for multiple functions#582
Conversation
Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
|
@copilot update the branch and move the changelog message to the top. |
Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
|
@copilot I updated the branch. Fix the failing pari formula tests (fast tests). Question to your change: why do we still need to keep the old code path with semicolon separator? Isn't this for 1 function only, where we don't need a separator anyway? |
- Update A001715, A002944, A077849, and A215020 to use multi-line format - Remove unnecessary semicolon separator for single functions (no separator needed when there's only one entry) Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
Fixed the failing tests in commit 88377bf. The merge incorrectly reverted some test entries (A001715, A002944, A077849, A215020) back to the old single-line format. Regarding your question about the semicolon separator: You're absolutely right! For single functions, there's no need for a separator since the formula generator produces a single entry. I've simplified the code to use an empty separator for single functions instead of the semicolon. |
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.