Skip to content

Support scientific notation in CSV output#33194

Merged
GiudGiud merged 2 commits into
idaholab:nextfrom
kyriv-lab:scientificNotation31150
Jul 1, 2026
Merged

Support scientific notation in CSV output#33194
GiudGiud merged 2 commits into
idaholab:nextfrom
kyriv-lab:scientificNotation31150

Conversation

@kyriv-lab

Copy link
Copy Markdown
Contributor

This PR resolves issue #31150

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 207127e1c3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/tests/outputs/csv/tests Outdated
@moosebuild

Copy link
Copy Markdown
Contributor

Job Precheck on 2f6aab7 : invalidated by @kyriv-lab

@kyriv-lab kyriv-lab marked this pull request as draft June 26, 2026 15:47
@moosebuild

Copy link
Copy Markdown
Contributor

All jobs on 2f6aab7 : invalidated by @kyriv-lab

@moosebuild

moosebuild commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Job Coverage, step Generate coverage on 2f6aab7 wanted to post the following:

Framework coverage

583127 #33194 2f6aab
Total Total +/- New
Rate 85.99% 85.99% +0.00% 100.00%
Hits 136176 136193 +17 21
Misses 22186 22188 +2 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

@moosebuild

moosebuild commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Job Documentation, step Docs: sync website on 2f6aab7 wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild

Copy link
Copy Markdown
Contributor

Job Conda moose linux on 2f6aab7 : invalidated by @kyriv-lab

@kyriv-lab kyriv-lab marked this pull request as ready for review July 1, 2026 13:58

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2f6aab7d16

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread framework/src/outputs/CSV.C
@kyriv-lab kyriv-lab requested a review from GiudGiud July 1, 2026 15:56
@kyriv-lab kyriv-lab force-pushed the scientificNotation31150 branch from ab8a906 to 2f6aab7 Compare July 1, 2026 16:29
@moosebuild

Copy link
Copy Markdown
Contributor

Job Test, step Results summary on 2f6aab7 wanted to post the following:

Framework test summary

Compared against 583127a in job civet.inl.gov/job/3942927.

Added tests

Test Time (s) Memory (MB)
outputs/csv.scientific_notation 0.67 90.02

Modules test summary

Compared against 583127a in job civet.inl.gov/job/3942927.

No change

@GiudGiud GiudGiud merged commit 28b9184 into idaholab:next Jul 1, 2026
57 checks passed
@kyriv-lab kyriv-lab deleted the scientificNotation31150 branch July 2, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants