Skip to content

Commit 20da91f

Browse files
sbryngelsonclaude
andcommitted
Fix mixed-precision stp description: half, not single
In mixed-precision mode (--mixed), stp is set to half_precision (kind=2), not single_precision, as confirmed in src/common/m_precision_select.f90. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 041cfa6 commit 20da91f

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.claude/rules/fortran-conventions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ These are caught by `./mfc.sh precheck` (source lint step 4/5):
3333
- `wp` (working precision): used for computation. Double by default.
3434
- `stp` (storage precision): used for I/O. Double by default.
3535
- In single-precision mode (`--single`): both become single.
36-
- In mixed-precision mode (`--mixed`): wp=double, stp=single.
36+
- In mixed-precision mode (`--mixed`): wp=double, stp=half.
3737
- MPI type matching: `mpi_p` must match `wp`, `mpi_io_p` must match `stp`.
3838
- Always use generic intrinsics: `sqrt` not `dsqrt`, `abs` not `dabs`.
3939
- Cast with `real(..., wp)` or `real(..., stp)`, never `dble(...)`.

CLAUDE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ Changes to `src/common/` affect ALL three executables. Test comprehensively.
154154
## Precision System
155155

156156
- `wp` = working precision (computation). `stp` = storage precision (I/O).
157-
- Default: both double. Single mode: both single. Mixed: wp=double, stp=single.
157+
- Default: both double. Single mode: both single. Mixed: wp=double, stp=half.
158158
- MPI types must match: `mpi_p``wp`, `mpi_io_p``stp`.
159159

160160
## Code Review Priorities

0 commit comments

Comments
 (0)