diag_manager: reword error message to make it more useful#1888
diag_manager: reword error message to make it more useful#1888uramirez8707 wants to merge 2 commits into
Conversation
uramirez8707
requested review from
bensonr,
rem1776 and
vithikashah001
as code owners
| call mpp_error(FATAL, "The axis for the variable:"//trim(field_yaml%get_var_varname())//& | ||
| "in the file: "//trim(this%get_file_fname())//" are not the same as the previously "//& | ||
| "defined variables. If using the cube sphere and defining the subregion with latlon "//& | ||
| "all variables in the file need to have the same x and y axis.") |
There was a problem hiding this comment.
Is it the same x & y axis or the same dimension lengths for the x & y axis?
There was a problem hiding this comment.
It has to be the same x and y axis.
There was a problem hiding this comment.
Is this what you mean? "Axes for variable {variable} in file {file} differ from the axes for the other variables in file {file}. All variables in a file need to be registered with the same axes"
There was a problem hiding this comment.
@mlee03 captured what I was trying to say. The difference between axis as a coordinate and axes as a proxy for the dimensions used is significant and should be better emphasized in the documentation.
There was a problem hiding this comment.
I like the updated message, but wanted to know if this applies only to subregional files?
3 participants
Description
reword error message to make it more useful
Fixes # (issue)
How Has This Been Tested?
ci
Checklist:
make distcheckpasses