bmi icon indicating copy to clipboard operation
bmi copied to clipboard

Add none location to get_var_location

Open mcflugen opened this issue 7 months ago • 3 comments

If a variable does not have a grid, then its location should be reported as "none". This means that get_var_grid for that variable will not be implemented or, if it is, the return value will be meaningless.

mcflugen avatar May 31 '25 23:05 mcflugen

@PhilMiller, @mdpiper I've added a note to alert users that the grid location of "none" is not formally part of the current version of BMI but that it may be added to future versions. Do you think this is a good idea? Maybe different wording and give a link to a (not yet existing) github issue where this is being discussed?

mcflugen avatar Jun 02 '25 18:06 mcflugen

Could we get a coherent treatment of scalar and none in a single proposal, rather than split between this and #184? For instance, why can't we just elaborate the specification of scalar to say that unlike node/face/edge it does not imply that get_var_grid will answer for it, rather than replacing scalar with none?

PhilMiller avatar Jun 02 '25 19:06 PhilMiller

Yes, we can do that. I've created #185 for this.

mcflugen avatar Jun 02 '25 19:06 mcflugen