Skip to content

Add support for GRTeclyn - #7

Merged
KAClough merged 5 commits into
mainfrom
enhancement/grteclyn_support
Jul 28, 2026
Merged

Add support for GRTeclyn#7
KAClough merged 5 commits into
mainfrom
enhancement/grteclyn_support

Conversation

@mirenradia

Copy link
Copy Markdown
Member

This PR adds the necessary modifications to TwoPunctures in order to enable integration with GRTeclyn.

I'll add more detail on the relevant GRTeclyn PR once I have opened that but they should be reviewed together.

I've also added a GitHub Actions workflow which mirrors the existing one for GRChombo. It just builds the BinaryBH example with TwoPunctures.

This is conditionally compiled on whether CH_SPACEDIM/AMREX_SPACEDIM is
defined.
This is currently required for AMReX's GNU Make build system.
This only tests building the BinaryBH example with TwoPunctures. It may
fail not do much initially since the TwoPunctures support has not yet
been merged into GRTeclyn.

Also rename GRChombo one.
@mirenradia
mirenradia requested a review from KAClough July 21, 2026 15:55
@mirenradia mirenradia self-assigned this Jul 21, 2026
@mirenradia mirenradia added the enhancement New feature or request label Jul 21, 2026
@github-project-automation github-project-automation Bot moved this to Todo in GRTeclyn Jul 21, 2026
@mirenradia mirenradia moved this from Todo to In Progress in GRTeclyn Jul 21, 2026

@KAClough KAClough left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks good to me and we have tried it and it works.

I will add the README info to the GRTeclyn documentation at some point soon.

@KAClough
KAClough merged commit 6f3cdf3 into main Jul 28, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in GRTeclyn Jul 28, 2026
@KAClough
KAClough deleted the enhancement/grteclyn_support branch July 28, 2026 16:50
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

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants