Skip to content

[WIP] Update urchin refinement to use last two coefficients in tail estimation#492

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/use-last-two-coefficients-tail-estimation
Closed

[WIP] Update urchin refinement to use last two coefficients in tail estimation#492
Copilot wants to merge 1 commit intomainfrom
copilot/use-last-two-coefficients-tail-estimation

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

  • Change n_tail_orders from 1 if warped_mesh.dim > 1 else 2 to 2 in warp_and_refine_until_resolved() in meshmode/mesh/generation.py
  • Run tests to validate the change (urchin generation succeeds)
Original prompt

This section details on the original issue you should resolve

<issue_title>Use the last two coefficients in tail estimation in urchin refinement</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@inducer inducer closed this Mar 19, 2026
Copilot AI requested a review from inducer March 19, 2026 15:14
Copilot stopped work on behalf of inducer due to an error March 19, 2026 15:14
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.

Use the last two coefficients in tail estimation in urchin refinement

2 participants