-
Notifications
You must be signed in to change notification settings - Fork 18
Remove text referencing scalar grid types #184
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
base: stable
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the documentation for BMI grid functions by removing references to a "scalar" grid type, ensuring clarity on variables that do not utilize an actual grid.
- Updated wording to reflect that exchange items may not be defined on a grid.
- Removed the "scalar" grid type from the list of valid grid types.
Comments suppressed due to low confidence (1)
docs/source/bmi.grid_funcs.md:55
- Ensure that the removal of the 'scalar' grid type is consistently reflected in any related documentation or usage examples for BMI grid functions.
- `scalar`
@@ -4,18 +4,12 @@ | |||
|
|||
The functions in this section describe {ref}`model grids <model-grids>`. | |||
In the BMI, | |||
every {term}`exchange item` is defined on a grid, | |||
if an {term}`exchange item` is defined on a grid, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider clarifying that not all exchange items are defined on a grid, and explicitly mention that variables without a grid will have their get_var_location return 'none'.
Copilot uses AI. Check for mistakes.
This is something that would only really be appropriate for a new major version of the standard, because it would make previously-conformant models that return |
Yeah, I should probably be merging this into the develop branch. Originally scalar variables were intended to work this way (and do for at least some BMI models) but the docs didn't reflect this. |
What was described as a "scalar" grid type in the docs was really a variable without a grid. The correct way to deal with this case is to not assign a grid to these variables (i.e. their
get_var_location
should return"none"
). None of theget_grid_
functions need to be implemented for these variables.