#2174: Test "Multi Patch Scalar Wave Equation" example
Reporter: Roland Haas
Status: open
Milestone: ET_2022_05
Version: development version
Type: task
Priority: major
Component: EinsteinToolkit website
Changes (by Roland Haas):
status: open (was closed)
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2174/test-multi-patch-…
#2655: Incorrect assert in Carpet/src/modes.cc
Reporter: Samuel Cupp
Status: resolved
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
Changes (by Samuel Cupp):
status: resolved (was new)
Comment (by Samuel Cupp):
Fix merged into master.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2655/incorrect-assert-…
#2655: Incorrect assert in Carpet/src/modes.cc
Reporter: Samuel Cupp
Status: new
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
Comment (by Roland Haas):
that would be very wrong. It should be:
```
assert(cctkGH->cctk_tile_min[d] == 0);
assert(cctkGH->cctk_tile_max[d] == cctkGH->cctk_lsh[d]);
```
as far as I can tell \(Carpet not actually supporting tiles\). Deleting is not an option.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2655/incorrect-assert-…
#2655: Incorrect assert in Carpet/src/modes.cc
Reporter: Samuel Cupp
Status: new
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
Comment (by Samuel Cupp):
@{557058:1671c5c3-29cc-4e83-9850-a152d33a6235} has suggested that perhaps it can simply be deleted. @{557058:59e031ba-9bb5-4298-a472-7b99d0ae6f22} @{557058:56049c54-f8c2-4b6c-9b88-ab697c967495} Do either of you have opinions on this?
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2655/incorrect-assert-…
#2655: Incorrect assert in Carpet/src/modes.cc
Reporter: Samuel Cupp
Status: new
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
In Carpet/src/modes.cc, there is a block of asserts
```
assert(cctkGH->cctk_lsh[d] >= 0);
assert(cctkGH->cctk_lsh[d] <= cctkGH->cctk_gsh[d]);
assert(cctkGH->cctk_lbnd[d] >= 0);
assert(cctkGH->cctk_lbnd[d] <= cctkGH->cctk_ubnd[d] + 1);
assert(cctkGH->cctk_ubnd[d] < cctkGH->cctk_gsh[d]);
assert(cctkGH->cctk_lbnd[d] + cctkGH->cctk_lsh[d] - 1 ==
cctkGH->cctk_ubnd[d]);
assert(cctkGH->cctk_lbnd[d] <= cctkGH->cctk_ubnd[d] + 1);
assert(cctkGH->cctk_lsh[d] <= cctkGH->cctk_ash[d]);
#ifdef CCTK_HAVE_CGH_TILE
assert(cctkGH->cctk_tile_min[d] == 0);
assert(cctkGH->cctk_tile_max[d] = cctkGH->cctk_lsh[d]);
#endif
assert(cctkGH->cctk_from[d] >= 0);
assert(cctkGH->cctk_from[d] <= cctkGH->cctk_to[d]);
assert(cctkGH->cctk_to[d] <= cctkGH->cctk_lsh[d]);
```
Inside the #ifdef, the second assert
```
assert(cctkGH->cctk_tile_max[d] = cctkGH->cctk_lsh[d]);
```
is setting the value instead of doing a comparison. This is definitely wrong and should be changed.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2655/incorrect-assert-…
#2639: CarpetLib internal error with PreSync and analysis thorns
Reporter: Samuel Cupp
Status: new
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
Comment (by Roland Haas):
Great. I will use the one you already included then.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2639/carpetlib-interna…
#2639: CarpetLib internal error with PreSync and analysis thorns
Reporter: Samuel Cupp
Status: new
Milestone:
Version: development version
Type: bug
Priority: major
Component: Carpet
Comment (by Samuel Cupp):
Roland, you asked for a parfile to reproduce the behavior, but it looks like I included one in the original ticket. To have the reads/writes for BaikalVacuum, you will want to checkout my scupp/BaikalRDWR branch of wvuthorns.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2639/carpetlib-interna…