Skip to content

Commit 9d78d61

Browse files
authored
Document removing the testing within mpas_xarray
Unit testing is performed elsewhere instead.
1 parent b78b3ea commit 9d78d61

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

design_docs/mpas_xarray_class.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,11 +110,15 @@ addressing any requirements. These include:
110110
- `_ensure_list`
111111
- `_get_datetimes`
112112
- `_remove_repeated_time_index`
113+
- I have removed the ability to run `mpas_xarray.py` as a script and the associated
114+
tests. This is on the premise that 1) the test were outdated and would have
115+
needed to be updated to work with the current code and 2) unit testing in
116+
`test/test_mpas_xarray.py` takes care of this capability in a better way.
113117
- I have tried to make variable names a bit more verbose in various places
114118
(e.g. dataset in place of ds).
115119
- I have tried to improve the docstrings using a syntax that should be useful
116120
for generating documentation later on.
117-
- I have update unit testing to work with the new inerface, notably the
121+
- I have update unit testing to work with the new inerface, notably the
118122

119123
<h2> Implementation: MPAS-Analysis specific functionality should be supported in
120124
xarray preprossing <br>

0 commit comments

Comments
 (0)