Skip to content

Examples: Remove particles.do_tiling#6809

Merged
ax3l merged 1 commit intoBLAST-WarpX:developmentfrom
ax3l:fix-tiling-defaults
Apr 28, 2026
Merged

Examples: Remove particles.do_tiling#6809
ax3l merged 1 commit intoBLAST-WarpX:developmentfrom
ax3l:fix-tiling-defaults

Conversation

@ax3l
Copy link
Copy Markdown
Member

@ax3l ax3l commented Apr 27, 2026

Set by default: on for CPU, off for GPU.

If set to on (1) on GPU runs, this asserts.

Isolated from #6801

Set by default: on for CPU, off for GPU.
@ax3l ax3l requested review from atmyers and lucafedeli88 April 27, 2026 18:09
@ax3l ax3l added the component: tests Tests and CI label Apr 27, 2026
@ax3l ax3l requested a review from clarkse April 27, 2026 18:18
@ax3l ax3l changed the title Examples: Remve particles.do_tiling Examples: Remove particles.do_tiling Apr 27, 2026
@ax3l ax3l added bug Something isn't working bug: affects latest release Bug also exists in latest release version labels Apr 27, 2026
Copy link
Copy Markdown
Member

@lucafedeli88 lucafedeli88 left a comment

Choose a reason for hiding this comment

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

Thanks, @ax3l !

Copy link
Copy Markdown
Member

@atmyers atmyers left a comment

Choose a reason for hiding this comment

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

Note that after AMReX-Codes/amrex#5268 Redistribute will no longer do this assert. But, I see no need to override the default in these scripts.

@ax3l ax3l merged commit bc30e14 into BLAST-WarpX:development Apr 28, 2026
47 of 48 checks passed
@ax3l ax3l deleted the fix-tiling-defaults branch April 28, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: affects latest release Bug also exists in latest release version bug Something isn't working component: tests Tests and CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants