Skip to content

Start the crosshairs on a cell center - #277

Merged
WeiqunZhang merged 1 commit into
AMReX-Codes:mainfrom
WeiqunZhang:fix-crosshair-cell-center
Sep 29, 2026
Merged

WeiqunZhang merged 1 commit into
AMReX-Codes:mainfrom
WeiqunZhang:fix-crosshair-cell-center

Conversation

@WeiqunZhang

Copy link
Copy Markdown
Member

Fixes #274. A default slice sat on the raw domain midpoint, which is a cell face when the cell count is even, so the crosshairs jumped to a cell center on the first Position step. The default now snaps to the finest-level sample holding the midpoint (a node for nodal data).

The navigation and mapped-grid smoke tests zoom or click around the exact midpoint, so they recenter the slices first.

Fixes AMReX-Codes#274. A default slice sat on the raw domain midpoint, which is a
cell face when the cell count is even, so the crosshairs jumped to a
cell center on the first Position step. The default now snaps to the
finest-level sample holding the midpoint (a node for nodal data).

The navigation and mapped-grid smoke tests zoom or click around the
exact midpoint, so they recenter the slices first.
@WeiqunZhang
WeiqunZhang merged commit f0e5437 into AMReX-Codes:main Sep 29, 2026
12 checks passed
@WeiqunZhang
WeiqunZhang deleted the fix-crosshair-cell-center branch September 29, 2026 21:19
@dpgrote

dpgrote commented Sep 29, 2026

Copy link
Copy Markdown

Thanks!

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.

Minor bug with crosshairs lines

2 participants