Merge pull request #826 from davidhassell/REVIEW

Remove REVIEW comments

Author: davidhassell

cf/cfimplementation.py

@@ -27,7 +27,6 @@
 )
 from .data import Data
 
-# REVIEW: h5: `cfimplementation.py`: import `CFAH5netcdfArray`, `CFANetCDF4Array`, `H5netcdfArray`,`NetCDF4Array`
 from .data.array import (
     BoundsFromNodesArray,
     CellConnectivityArray,
@@ -116,7 +115,6 @@ def set_construct(self, parent, construct, axes=None, copy=True, **kwargs):
             parent, construct, axes=axes, copy=copy, **kwargs
         )
 
-    # REVIEW: h5: `initialise_CFANetCDF4Array`: new method to initialise `CFANetCDF4Array`
     def initialise_CFANetCDF4Array(self, **kwargs):
         """Return a `CFANetCDF4Array` instance.
 
@@ -133,7 +131,6 @@ def initialise_CFANetCDF4Array(self, **kwargs):
         cls = self.get_class("CFANetCDF4Array")
         return cls(**kwargs)
 
-    # REVIEW: h5: `initialise_CFAH5netcdfArray`: new method to initialise `CFAH5netcdfArray`
     def initialise_CFAH5netcdfArray(self, **kwargs):
         """Return a `CFAH5netcdfArray` instance.
 

cf/constants.py

@@ -63,7 +63,6 @@
     "LOG_LEVEL": logging.getLevelName(logging.getLogger().level),
     "BOUNDS_COMBINATION_MODE": "AND",
     "CHUNKSIZE": parse_bytes(_CHUNKSIZE),
-    # REVIEW: active: `CONSTANTS`: new constants 'active_storage', 'active_storage_url', 'active_storage_max_requests'
     "active_storage": False,
     "active_storage_url": None,
     "active_storage_max_requests": 100,

cf/data/array/__init__.py

@@ -1,19 +1,11 @@
 from .boundsfromnodesarray import BoundsFromNodesArray
 from .cellconnectivityarray import CellConnectivityArray
-
-# REVIEW: h5: `__init__.py`: import `CFAH5netcdfArray`
 from .cfah5netcdfarray import CFAH5netcdfArray
-
-# REVIEW: h5: `__init__.py`: import `CFAH5netcdfArray`
 from .cfanetcdf4array import CFANetCDF4Array
 from .fullarray import FullArray
 from .gatheredarray import GatheredArray
-
-# REVIEW: h5: `__init__.py`: import `H5netcdfArray`
 from .h5netcdfarray import H5netcdfArray
 from .netcdfarray import NetCDFArray
-
-# REVIEW: h5: `__init__.py`: import `NetCDF4Array`
 from .netcdf4array import NetCDF4Array
 from .pointtopologyarray import PointTopologyArray
 from .raggedcontiguousarray import RaggedContiguousArray

cf/data/array/cfah5netcdfarray.py

@@ -1,4 +1,3 @@
-# REVIEW: h5: `CFAH5netcdfArray`: New class for accessing CFA with `h5netcdf`
 from .h5netcdfarray import H5netcdfArray
 from .mixin import CFAMixin
 

cf/data/array/cfanetcdf4array.py

@@ -1,4 +1,3 @@
-# REVIEW: h5: `CFAnetCDF4Array`: New class for accessing CFA with `netCDF4`
 from .mixin import CFAMixin
 from .netcdf4array import NetCDF4Array
 

cf/data/array/fullarray.py

@@ -16,7 +16,6 @@ class FullArray(IndexMixin, Array):
 
     """
 
-    # REVIEW: h5: `__init__`: replace units/calendar API with attributes
     def __init__(
         self,
         fill_value=None,
@@ -121,7 +120,6 @@ def __str__(self):
 
         return f"Filled with {fill_value!r}"
 
-    # REVIEW: getitem: `_get_array`: new method to convert subspace to numpy array
     def _get_array(self, index=None):
         """Returns the full array.
 
@@ -156,7 +154,6 @@ def _get_array(self, index=None):
 
         return array
 
-    # REVIEW: getitem: `array`: New property to convert subspace to numpy array
     @property
     def array(self):
         """Return an independent numpy array containing the data.

cf/data/array/h5netcdfarray.py

@@ -1,4 +1,3 @@
-# REVIEW: h5: `H5netcdfArray`: New class to access netCDF with `h5netcdf`
 import cfdm
 
 from ...mixin_container import Container
@@ -49,8 +48,6 @@ def _lock(self):
         """
         return netcdf_lock
 
-    # REVIEW: h5: `_get_array`: Ignore this for h5 review
-    # REVIEW: getitem: `_get_array`: new method to convert subspace to numpy array.
     def _get_array(self, index=None):
         """Returns a subspace of the dataset variable.
 

cf/data/array/locks.py

@@ -1,4 +1,3 @@
-# REVIEW: h5: `locks.py`:  New module to provide file locks
 from dask.utils import SerializableLock
 
 # Global lock for netCDF file access

cf/data/array/mixin/__init__.py

@@ -1,9 +1,6 @@
-# REVIEW: active: `__init__.py`: import `ActiveStorageMixin`
 from .activestoragemixin import ActiveStorageMixin
 from .arraymixin import ArrayMixin
 from .cfamixin import CFAMixin
 from .compressedarraymixin import CompressedArrayMixin
 from .filearraymixin import FileArrayMixin
-
-# REVIEW: getitem: `__init__.py`: import `IndexMixin`
 from .indexmixin import IndexMixin

cf/data/array/mixin/activestoragemixin.py

@@ -1,6 +1,3 @@
-# REVIEW: active: `ActiveStorageMixin`: new mixin class `ActiveStorageMixin`
-
-
 class ActiveStorageMixin:
     """Mixin class for enabling active storage operations.