Skip to content

Update energy line test cfg so that pfv fields are also saved#1256

Merged
fso42 merged 1 commit intomasterfrom
updateEnergyCfg
Mar 19, 2026
Merged

Update energy line test cfg so that pfv fields are also saved#1256
fso42 merged 1 commit intomasterfrom
updateEnergyCfg

Conversation

@awirb
Copy link
Copy Markdown
Contributor

@awirb awirb commented Mar 18, 2026

this is required for the figures and due to the recent change in the export of time steps and final timesteps for resTypes the pfv field was not available anymore with the default settings

@awirb awirb requested a review from fso42 March 18, 2026 14:02
@awirb awirb added the confirmed Something isn't working label Mar 18, 2026
@qltysh
Copy link
Copy Markdown
Contributor

qltysh bot commented Mar 18, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@fso42 fso42 added the bugfix label Mar 18, 2026
@fso42 fso42 merged commit 58a9be8 into master Mar 19, 2026
4 checks passed
@fso42 fso42 deleted the updateEnergyCfg branch March 19, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix confirmed Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants