Skip to content

New ways to define a test_diagnosis_t object#126

Merged
rouson merged 2 commits intomainfrom
assign-logical-to-test_diagnosis_t
Nov 9, 2025
Merged

New ways to define a test_diagnosis_t object#126
rouson merged 2 commits intomainfrom
assign-logical-to-test_diagnosis_t

Conversation

@rouson
Copy link
Copy Markdown
Contributor

@rouson rouson commented Nov 3, 2025

This PR supports defining a test_diagnosis_t object by

  1. Assigning a .logical. right-hand side, e.g., test_diagnosis = x > y or
  2. Hardwiring a passing result via a new function test_diagnosis = passing_test().

@rouson rouson requested a review from ktras November 3, 2025 20:53
@rouson rouson changed the title New ways to define a test_diagnosis_t New ways to define a test_diagnosis_t object Nov 4, 2025
Copy link
Copy Markdown
Collaborator

@ktras ktras left a comment

Choose a reason for hiding this comment

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

LGTM except for a minor typo in a comment. Feel free to merge after accepting the suggested typo fix.

Comment thread src/julienne/julienne_test_diagnosis_m.F90 Outdated
rouson and others added 2 commits November 8, 2025 10:16
This commit adds
1. A passing_test() function that constructs a test_diagnosis_t
   result corresponding to a passing test
2. A test_diagnosis_t defined assignment with a logical right-hand
   side that determines whether the test passes or fails.
Both new procedures define an object with a zero-length diagnostics
string.
Co-authored-by: Katherine Rasmussen <krasmussen@lbl.gov>
@rouson rouson force-pushed the assign-logical-to-test_diagnosis_t branch from ae018ed to f1ba64e Compare November 8, 2025 18:30
@rouson rouson merged commit e532d17 into main Nov 9, 2025
102 of 103 checks passed
@rouson rouson deleted the assign-logical-to-test_diagnosis_t branch November 9, 2025 19:25
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.

2 participants