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.

Sign up for GitHub

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

Jump to bottom

printing NA in mse_tables #463

Closed
yairchn opened this issue Oct 27, 2021 · 0 comments · Fixed by #464
Closed

printing NA in mse_tables #463

yairchn opened this issue Oct 27, 2021 · 0 comments · Fixed by #464
Assignees
@charleskawczynski

Comments

@yairchn
Copy link
Member

yairchn commented Oct 27, 2021

the Not Applicable in use tables prints as NA but when copy pasted needs to be "NA" therefore a simple copy pasting of the tables would not work

TurbulenceConvection.jl/integration_tests/utils/mse_tables.jl

Line 17 in 4243d5e

all_best_mse["ARM_SGP"]["qi_mean"] = "NA"

@charleskawczynski charleskawczynski mentioned this issue Oct 27, 2021
Merged
bors bot added a commit that referenced this issue Oct 27, 2021
@bors @charleskawczynski
Merge #464
5b2b0c9 
464: Fix NA in print_new_mse r=charleskawczynski a=charleskawczynski

Closes #463. Thanks for catching this! This should have been incorporated in #459 / #460.

Co-authored-by: Charles Kawczynski <[email protected]>
bors bot added a commit that referenced this issue Oct 27, 2021
@bors @charleskawczynski
Merge #464
293853a 
464: Fix NA in print_new_mse r=charleskawczynski a=charleskawczynski

Closes #463. Thanks for catching this! This should have been incorporated in #459 / #460.

Co-authored-by: Charles Kawczynski <[email protected]>
@bors bors bot closed this as completed in 091e9e3 Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@charleskawczynski charleskawczynski

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix NA in print_new_mse CliMA/TurbulenceConvection.jl
2 participants
@charleskawczynski @yairchn