Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DIAGNO] add an option to write a more accurate output file for the afield_points diagnostic #237

Merged
merged 7 commits into from
May 17, 2024

Conversation

jonathanschilling
Copy link
Collaborator

[Similar as #234, but this time for the new afield_points diagnostic. Started working on that separately and wanted to wait for feedback on 234, thus only submitting now.]

With this patch, the user can use a DIAGNO_IN namelist flag lapoints_accurate_output to activate a different output file format for the diagno_atest.<runId> output file, which has:

  • the number of points that were written in the first line (for easier parsing, similar to the fluxloop and segmented Rogowski output files) and
  • more digits of accuracy for each of the outputs (the output format was copied from the corresponding writing routine for the segmented Rogowski sensors)

By default, this option is disabled in order to retain backwards compatibility.

Also, added writing of the flags lapoints_accurate_output and lbpoints_accurate_output to write_diagno_input.

Copy link
Collaborator

@lazersos lazersos left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@lazersos lazersos merged commit 48dcb0d into develop May 17, 2024
2 checks passed
@lazersos lazersos deleted the diagno-afield-more-digits-in-output branch May 17, 2024 11:40
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