Trac May 2026

trac@lists.einsteintoolkit.org

1 participants
105 discussions

#2941: CCE_Export uses `setiosflags` unqualified without an explicit using-declaration
by Maxwell Rizzo 12 May '26

12 May '26

#2941: CCE_Export uses `setiosflags` unqualified without an explicit using-declaration Reporter: Maxwell Rizzo Status: new Milestone: Version: Type: bug Priority: minor Component: EinsteinToolkit thorn Changes (by Maxwell Rizzo): In line 169-170 of `h5_export.cc` ```cpp basename << base_file_name << "R" << setiosflags(ios::fixed) << setprecision(2) << rad << "." << extension; ``` and the only using statement on line 13 ```cpp using std::string, std::ostringstream, std::map, std::ios, std::setprecision; ``` `std::setiosflags` is missing from the using-declaration in order to be used unqualified. Corresponding [PR](https://github.com/deborahferguson/CCE_Export/pull/17) -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2941/cce_export-uses-s…

1 0

#2941: CCE_Export uses `setiosflags` unqualified without an explicit using-declaration
by Maxwell Rizzo 12 May '26

12 May '26

#2941: CCE_Export uses `setiosflags` unqualified without an explicit using-declaration Reporter: Maxwell Rizzo Status: new Milestone: Version: Type: bug Priority: minor Component: EinsteinToolkit thorn In line 169-170 of `h5_export.cc` ```cpp basename << base_file_name << "R" << setiosflags(ios::fixed) << setprecision(2) << rad << "." << extension; ``` and the only using statement on line 13 ```cpp using std::string, std::ostringstream, std::map, std::ios, std::setprecision; ``` `std::setiosflags` is missing from the using-declaration if will be used unqualified. Corresponding [PR](https://github.com/deborahferguson/CCE_Export/pull/17) -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2941/cce_export-uses-s…

1 0

#2940: CCE_Export: allow for changing array_size
by Roland Haas 12 May '26

12 May '26

#2940: CCE_Export: allow for changing array_size Reporter: Roland Haas Status: new Milestone: Version: Type: bug Priority: minor Component: Comment (by Roland Haas): Pull request is here: https://github.com/deborahferguson/CCE_Export/pull/16 -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2940/cce_export-allow-…

1 0

#2940: CCE_Export: allow for changing array_size
by Roland Haas 12 May '26

12 May '26

#2940: CCE_Export: allow for changing array_size Reporter: Roland Haas Status: new Milestone: Version: Type: bug Priority: minor Component: remove static variable whose size depends on a function argument -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2940/cce_export-allow-…

1 0

#2939: AHFinderDirect: range options for track_origin_source_x/y/z
by Jordan Nicoules 08 May '26

08 May '26

#2939: AHFinderDirect: range options for track_origin_source_x/y/z Reporter: Jordan Nicoules Status: new Milestone: Version: Type: bug Priority: minor Component: EinsteinToolkit thorn In `AHFinderDirect` param.ccl there is (same for y, z): ``` string track_origin_source_x[101] "grid scalar containing the x component of the origin estimate" STEERABLE=recover { "" :: "don't use this feature" "[a-zA-Z_][a-zA-Z0-9_]*[:][:][a-zA-Z_][a-zA-Z0-9_]*(\[0-9+\])" :: "name of a grid scalar" } "" ``` While mimicking this and testing in a thorn of my own, I realized that `""` matches everything, so the second range is basically useless. **Is it what is meant by** `"don't use this feature"`? I think I rather understand it as "do not put nothing". If this is not the intention, first something should be done about the empty string and default value. What's more, the range is actually incorrect (see explanation from GPT-5.4 in Copilot below; I'll let you judge the validity of the explanation, you know that better than me). Possible fixes: 1. For the default: for example use a `"None"` value, with some catch in `AHFinderDirect_setup()` that checks the variable name and returns an error. With a quick test, an unknown/wrong variable name would trigger an error somewhere in the execution of `AHFinderDirect_find_horizons()` anyway. 2. For the actual range: * `^.*[:][:].*$` is used in `NSTracker` for example. * to keep a strict pattern, ``` "[a-zA-Z_][a-zA-Z0-9_]*[:][:][a-zA-Z_][a-zA-Z0-9_]*" :: "name of a grid scalar without index" "[a-zA-Z_][a-zA-Z0-9_]*[:][:][a-zA-Z_][a-zA-Z0-9_]*\[[0-9][0-9]*\]" :: "name of a grid scalar with index" ``` (GPT's recommendation to separate the two patterns for clarity and portability, instead of trying to match both at the same time). # Conclusion # I don't know if there are risks of breaking internal mechanics of AHFinderDirect by changing this. I would say that the current harm is mostly about false, or misleading, pretenses, as well as a bad template for other people like me. The possible harm of fixing is to break something else in AHFinderDirect by introducing a different default (this needs verification, but it doesn't look like it from a quick test). There may also be some backwards-compatibility issues if some people managed to use grid scalars that don't match the new pattern. ## GPT's explanation >Yes. The old regex from param.ccl is not actually matching PunctureTracker::pt_loc_y[0] as written. > >The key point is that Cactus string ranges use POSIX regexes, and the docs explicitly say extended regex support is not portable. In practice, you should write these as portable POSIX-BRE-style patterns, not PCRE-ish ones. > >Your old pattern is: > > `[a-zA-Z_][a-zA-Z0-9_]*[:][:][a-zA-Z_][a-zA-Z0-9_]*(\[0-9+\])` > >Why it fails: > >`\[` and `\]` correctly mean literal [ and ]. >But 0-9+ is outside a character class, so it means the literal characters 0, -, 9, +, not “one or more digits”. >The surrounding ( ) are also suspicious, because in POSIX basic regex they are literals unless escaped as`$` and `$`. So they are not doing grouping in the portable interpretation. So that pattern is effectively trying to match something like: > > `thorn::var([0-9+])` > >or at best a very literal bracket payload, not `thorn::var[0]`. > >That is why `""` made it “work”: per the Cactus docs, the empty regex matches anything. So before, validation was effectively bypassed. > >What you want, portably, is something like: > > `[a-zA-Z_][a-zA-Z0-9_]*[:][:][a-zA-Z_][a-zA-Z0-9_]*\[[0-9][0-9]*\]` > >That matches: > > * implementation name > * :: > * variable name > * [ digits ] > >including `PunctureTracker::pt_loc_y[0]`. -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2939/ahfinderdirect-ra…

1 0

#2931: Claude AI edits to TwoPuncturesX
by Steven R. Brandt 07 May '26

07 May '26

#2931: Claude AI edits to TwoPuncturesX Reporter: Steven R. Brandt Status: new Milestone: ET_2026_05 Version: Type: bug Priority: major Component: Comment (by Steven R. Brandt): This PR addresses the issue: https://bitbucket.org/einsteintoolkit/einsteininitialdata/pull-requests/22 But through a new thorn, not SpacetimeX/TwoPuncturesX. -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2931/claude-ai-edits-t…

1 0

#2733: TwoPunctures contains globally visible symbols not prefixed by thorn name
by Steven R. Brandt 07 May '26

07 May '26

#2733: TwoPunctures contains globally visible symbols not prefixed by thorn name Reporter: Roland Haas Status: new Milestone: Version: Type: bug Priority: minor Component: Comment (by Steven R. Brandt): This PR addresses the issue: https://bitbucket.org/einsteintoolkit/einsteininitialdata/pull-requests/22 -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2733/twopunctures-cont…

1 0

#2937: TwoPuncturesX largely duplicates TwoPunctures
by Steven R. Brandt 07 May '26

07 May '26

#2937: TwoPuncturesX largely duplicates TwoPunctures Reporter: Zach Etienne Status: new Milestone: Version: Type: bug Priority: major Component: Comment (by Steven R. Brandt): This PR addresses the issue: https://bitbucket.org/einsteintoolkit/einsteininitialdata/pull-requests/22 -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2937/twopuncturesx-lar…

1 0

#2936: CarpetX tsv output does not append files
by Roland Haas 07 May '26

07 May '26

#2936: CarpetX tsv output does not append files Reporter: Maxwell Rizzo Status: new Milestone: Version: Type: enhancement Priority: minor Component: CarpetX Comment (by Roland Haas): Pull request is here: https://github.com/EinsteinToolkit/CarpetX/pull/380 -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2936/carpetx-tsv-outpu…

1 0

#2929: GRHayLET/IllinoisGRMHD convert_IllinoisGRMHD_to_HydroBase schedule compatibility with VolumeIntegrals_*
by Maxwell Rizzo 07 May '26

07 May '26

#2929: GRHayLET/IllinoisGRMHD convert_IllinoisGRMHD_to_HydroBase schedule compatibility with VolumeIntegrals_* Reporter: Maxwell Rizzo Status: resolved Milestone: ET_2026_05 Version: ET_2025_05 Type: bug Priority: major Component: EinsteinToolkit thorn Changes (by Maxwell Rizzo): status: resolved (was new) -- Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2929/grhaylet-illinois…

1 0

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Trac May 2026