CPLHIST files assign standard_name to variables, but it isn't a CF-Convention standard name -- should it do that?
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 40/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- fortran
- Domain
- data
Research direction
The issue asks whether the standard_name attribute written to CPLHIST output variables should be removed, so start by searching the Fortran sources for where standard_name is set on those variables. Confirm with a maintainer that removal is wanted before editing, since the reporter is asking whether they are missing a reason to keep it. Done means either the attribute is removed and the CF compliance check no longer flags these names, or the reason to keep it is documented on the issue.
Written by the indexing model from the issue text.
Description
CPLHIST files add the standard_name attribute to each output variable. But, it's the same name as the variable name, and hence is redundant. But, the naming convention follows the CMEPS naming standard -- but not the CF-Convention standard_name. So checking for compliance with CF just has it report the names as an error.
I'm thinking it doesn't serve enough of a purpose and should just be removed. But, am I missing something?
- Dominant language
- Fortran
- Stars
- 36
- Forks
- 107
- Avg merge
- 1d 5h
- Merged PRs (30d)
- 2
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ESCOMP/CMEPS
-
Latent heat bug in COARE atm/ocn flux routine for CESMMay be free again A pull request for this issue was closed without being merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 68/100
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 45/100
-
enhancement
Difficulty 3/5 Half a day Newbie friendliness 50/100
Similar issues
-
[Anon] Imperium - Adeptus Astartes - Dark Angels: Azrael is missing the Lion Helm wargear abilityOpen
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
-
Grain de MaliceOpennew-spider
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
alltheplaces/alltheplaces#19646 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 1/5 Under an hour Newbie friendliness 74/100
BSData/age-of-sigmar-4th#1393 ·