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

Documentation of output_forward_op_errors #530

Closed
kdraeder opened this issue Aug 18, 2023 · 3 comments · Fixed by #605
Closed

Documentation of output_forward_op_errors #530

kdraeder opened this issue Aug 18, 2023 · 3 comments · Fixed by #605
Assignees
Labels
Documentation Improvements or additions to documentation

Comments

@kdraeder
Copy link
Contributor

kdraeder commented Aug 18, 2023

Describe the bug

filter_mod.f90:

subroutine verbose_forward_op_output(qc_ens_handle, prior_post, ens_size, keys)

subroutine verbose_forward_op_output:
do i = 1, ens_size
   do j = 1, qc_ens_handle%my_num_vars
      if(nint(qc_ens_handle%copies(i, j)) /= 0) write(forward_unit, *) i, keys(j), nint(qc_ens_handle%copies(i, j))
   end do
end do

So ensemble member is printed first, then (local) observation number.

But filter_mod.rst says "Each line will have three values listed: the observation number,
the ensemble member number, and the istatus return code."

| | | this file. Each line will have three |

Error Message

None. User might notice that the numbers in the first column range from 1 to ens_size and be skeptical that it's a coincidence.

Which model(s) are you working with?

Any

Version of DART

v10.8.2-2-g8c29b618c

Have you modified the DART code?

No

Build information

All

@kdraeder kdraeder added Documentation Improvements or additions to documentation Trivial Does not affect functionality or data, does not need a workaround, but would make Tim happy. labels Aug 18, 2023
@kdraeder kdraeder self-assigned this Aug 18, 2023
@hkershaw-brown
Copy link
Member

Hi @kdraeder Nice catch.

This is just a note for git information, this output from git describe --tags
v10.8.2-2-g8c29b618c

Tells us you are using
Tag v10.8.2 with 2 commits that you've added since tag v10.8.2. In this case I believe you've run git describe --tags while having cam-fv_small_fixes checked out.

@kdraeder
Copy link
Contributor Author

kdraeder commented Aug 18, 2023 via email

@hkershaw-brown hkershaw-brown mentioned this issue Aug 18, 2023
12 tasks
@hkershaw-brown hkershaw-brown assigned ann-norcio and unassigned kdraeder Oct 4, 2023
@hkershaw-brown hkershaw-brown removed the Trivial Does not affect functionality or data, does not need a workaround, but would make Tim happy. label Oct 4, 2023
@hkershaw-brown
Copy link
Member

hkershaw-brown commented Oct 4, 2023

@ann-norcio this a documentation fix, filter_mod.rst needs changing to correctly describe what the code outputs.

To build the dart documentation on your machine you'll need to install sphinx
https://ncar.github.io/dart-developers-guide/documentation.html#installing-sphinx-locally

Sphinx==6.2.1
sphinx-rtd-theme==1.2.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants