Skip to content

t8code July release#127

Merged
benegee merged 14 commits into
mainfrom
bg/t8-26-7
Jul 8, 2026
Merged

t8code July release#127
benegee merged 14 commits into
mainfrom
bg/t8-26-7

Conversation

@benegee

@benegee benegee commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Closes #130.

@benegee benegee marked this pull request as ready for review July 7, 2026 09:02
@benegee

benegee commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator Author

Ready for review @spenke91

In the end, this was quite lightweight.
I added two lines to fixes.sh to get the scheme for t8_scheme_new_default() and t8_eclass_scheme_is_default(scheme, eclass) right. The is rooted in the include sequence in t8code, could be avoided by adding some forward declaration, but never caused trouble so far.

I had to manually tweak the generation because in the current release some header end up in a new folder PRIVATE_HEADER. xref DLR-AMR/t8code#2355

@spenke91 spenke91 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your efforts @benegee ! To a Julia newbie like me, it looks all fine 👍

Comment thread dev/Artifacts.toml
Comment thread src/Libt8.jl
@benegee benegee merged commit 97c7668 into main Jul 8, 2026
18 checks passed
@benegee benegee deleted the bg/t8-26-7 branch July 8, 2026 14:38
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