Merge pull request #229 from yarikoptic:enh-codespell

PiperOrigin-RevId: 750271007
Change-Id: I920086dc83dd6d20f06b9addb8abb1ec0fefe23a

Author: copybara-github

bazelisk.py

@@ -170,18 +170,19 @@ def read_remote_text_file(url):
 
 
 def get_version_history(bazelisk_directory):
-    return sorted(
-        (
-            release["tag_name"]
-            for release in get_releases_json(bazelisk_directory)
-            if not release["prerelease"]
-        ),
-        # This only handles versions with numeric components, but that is fine
-        # since prerelease verisons have been excluded.
-        key=lambda version: tuple(int(component)
-                                  for component in version.split('.')),
-        reverse=True,
-    )
+  return sorted(
+      (
+          release["tag_name"]
+          for release in get_releases_json(bazelisk_directory)
+          if not release["prerelease"]
+      ),
+      # This only handles versions with numeric components, but that is fine
+      # since prerelease versions have been excluded.
+      key=lambda version: tuple(
+          int(component) for component in version.split(".")
+      ),
+      reverse=True,
+  )
 
 
 def resolve_latest_version(version_history, offset):

docs/conf.py

@@ -363,7 +363,9 @@ def _python_type_to_xref(
         'desc': 'C++ class',
     },
     'absl::StatusOr': {
-        'url': 'https://abseil.io/docs/cpp/guides/statuss#returning-a-status-or-a-value',
+        'url': (
+            'https://abseil.io/docs/cpp/guides/status#returning-a-status-or-a-value'
+        ),
         'object_type': 'class',
         'desc': 'C++ class',
     },
@@ -378,22 +380,30 @@ def _python_type_to_xref(
         'desc': 'C++ enumeration',
     },
     'absl::Time': {
-        'url': 'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime',
+        'url': (
+            'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime'
+        ),
         'object_type': 'class',
         'desc': 'C++ class',
     },
     'absl::InfiniteFuture': {
-        'url': 'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime',
+        'url': (
+            'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime'
+        ),
         'object_type': 'function',
         'desc': 'C++ function',
     },
     'absl::InfinitePast': {
-        'url': 'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime',
+        'url': (
+            'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime'
+        ),
         'object_type': 'function',
         'desc': 'C++ function',
     },
     'absl::Now': {
-        'url': 'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime',
+        'url': (
+            'https://abseil.io/docs/cpp/guides/time#absolute-times-with-absltime'
+        ),
         'object_type': 'function',
         'desc': 'C++ function',
     },
@@ -403,12 +413,16 @@ def _python_type_to_xref(
         'desc': 'C++ class',
     },
     'absl::Cord': {
-        'url': 'https://github.com/abseil/abseil-cpp/blob/master/absl/strings/cord.h',
+        'url': (
+            'https://github.com/abseil/abseil-cpp/blob/master/absl/strings/cord.h'
+        ),
         'object_type': 'class',
         'desc': 'C++ class',
     },
     'absl::AnyInvocable': {
-        'url': 'https://github.com/abseil/abseil-cpp/blob/master/absl/functional/any_invocable.h',
+        'url': (
+            'https://github.com/abseil/abseil-cpp/blob/master/absl/functional/any_invocable.h'
+        ),
         'object_type': 'class',
         'desc': 'C++ class',
     },

docs/doctest_test.py

@@ -16,7 +16,7 @@
 This allows doctest examples to be conveniently updated in case of changes to
 the output format.
 
-After commiting or staging changes, you can run this with the `--in-place`
+After committing or staging changes, you can run this with the `--in-place`
 option and then inspect the diff.
 
 This supports top-level `await` in tests, since that provides a convenient way
@@ -327,7 +327,7 @@ def _ast_asyncify(code: str, wrapper_name: str) -> ast.Module:
   This is derived from a similar workaround in IPython.
 
   Args:
-    code: Python source code to pase.
+    code: Python source code to pass.
     wrapper_name: Name to use for function.
 
   Returns:

docs/environment.rst

@@ -96,7 +96,7 @@ Debugging
    Enables debug logging for tensorstore internal subsystems.  Set to comma
    separated list of values, where each value is one of ``name=int`` or just
    ``name``. When ``all`` is, present, then verbose logging will be enabled for
-   each subsytem, otherwise logging is set only for those subsystems present in
+   each subsystem, otherwise logging is set only for those subsystems present in
    the list.
 
    Verbose flag values include: ``curl``, ``distributed``, ``file``,

docs/installation.rst

@@ -264,7 +264,7 @@ The following CMake generators are supported:
 - Ninja and Ninja Multi-Config
 - Makefile generators
 - Visual Studio generators
-- Xcode (targetting arm64 only)
+- Xcode (targeting arm64 only)
 
 The Ninja generator is recommended because it provides the fastest builds.
 

docs/python/indexing.rst

@@ -423,7 +423,7 @@ Ellipsis
 
 Specifying the special `Ellipsis` value (:python:`...`) is equivalent
 to specifying as many full slices :python:`:` as needed to consume the
-remaining dimensions of the original domin not consumed by other
+remaining dimensions of the original domain not consumed by other
 indexing terms:
 
 .. doctest::

docs/schema_schema.yml

@@ -80,7 +80,7 @@ definitions:
       - A numerical :literal:`multiplier`, represented as a `double-precision
         floating-point number
         <https://en.wikipedia.org/wiki/Double-precision_floating-point_format>`_.
-        A multiplier of ``1`` may be used to indicate a quanity equal to a
+        A multiplier of ``1`` may be used to indicate a quantity equal to a
         single base unit.
 
       - A :literal:`base_unit`, represented as a string.  An empty string may be used

examples/BUILD

@@ -63,11 +63,12 @@ tensorstore_cc_binary(
     srcs = ["image_convolution.cc"],
     deps = [
         "//tensorstore:array",
+        "//tensorstore:box",
+        "//tensorstore:contiguous_layout",
+        "//tensorstore:data_type",
         "//tensorstore:index",
-        "//tensorstore/index_space:dim_expression",
-        "//tensorstore/index_space:transformed_array",
         "//tensorstore/util:iterate_over_index_range",
-        "//tensorstore/util:status",
+        "//tensorstore/util:span",
         "@com_google_absl//absl/functional:function_ref",
     ],
 )

examples/image_convolution.cc

@@ -12,16 +12,23 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <stddef.h>
+
 #include <algorithm>
+#include <array>
+#include <cassert>
+#include <cstring>
 #include <iostream>
+#include <utility>
 
 #include "absl/functional/function_ref.h"
 #include "tensorstore/array.h"
+#include "tensorstore/box.h"
+#include "tensorstore/contiguous_layout.h"
+#include "tensorstore/data_type.h"
 #include "tensorstore/index.h"
-#include "tensorstore/index_space/dim_expression.h"
-#include "tensorstore/index_space/transformed_array.h"
 #include "tensorstore/util/iterate_over_index_range.h"
-#include "tensorstore/util/status.h"
+#include "tensorstore/util/span.h"
 
 using ::tensorstore::Index;
 
@@ -34,7 +41,7 @@ tensorstore::SharedArray<int, 2> ApplyKernel(
     const tensorstore::ArrayView<const int, 2> in,
     const tensorstore::ArrayView<const double, 2> kernel) {
   // Compute bounds for the offset.
-  // FIXME: It's akward that we cannot do this:
+  // FIXME: It's awkward that we cannot do this:
   //   std::array<Index, 2> k(kernel.shape());
   //
   std::array<Index, 2> k;
@@ -80,7 +87,7 @@ tensorstore::SharedArray<int, 2> ApplyKernel(
               }
             });
 
-        // Again, the most intutive way to write an array value
+        // Again, the most intuitive way to write an array value
         // is not permitted:
         //
         //   dest[indices] = (sum / count);  // error: no viable overloaded '='
@@ -154,7 +161,7 @@ void AffineWarpGrid(size_t xmax, size_t ymax,
 }
 
 // AffineWarpInverseGrid computes the inverse mapping from AffineWarpGrid,
-// so it can be used to map from a destination image to a souce image.
+// so it can be used to map from a destination image to a source image.
 void AffineWarpInverseGrid(size_t xmax, size_t ymax,
                            tensorstore::span<const double, 6> M,
                            AffineWarpGridFunction fn) {
@@ -237,7 +244,7 @@ void PrintCSVArray(tensorstore::ArrayView<T, N> data) {
   // reference for every element.
   //
   // There is a streaming operator already, but the output is
-  // this is equvalent to:
+  // this is equivalent to:
   // for (int x = 0; x < data.shape()[0]; x++)
   //  for (int y = 0; y < data.shape()[1]; y++) {
   //     ... body ...

python/tensorstore/keyword_arguments.h

@@ -23,7 +23,7 @@
 /// This mechanism allows individual keyword arguments to be defined as
 /// `ParamDef` types, that specifies the name, documentation, argument type, and
 /// operation to perform to "apply" the argument.  These `ParamDef` types can
-/// then be re-used by multiple pybind11 functions while avoiding duplication.
+/// then be reused by multiple pybind11 functions while avoiding duplication.
 ///
 /// Each `ParamDef` type should be a struct with the following members, and no
 /// non-static data members (i.e. it should be empty):
@@ -52,12 +52,13 @@
 #include <pybind11/pybind11.h>
 // Other headers must be included after pybind11 to ensure header-order
 // inclusion constraints are satisfied.
+#include <string>
 #include <string_view>
 
 #include "absl/strings/ascii.h"
 #include "absl/strings/str_split.h"
 #include "python/tensorstore/status.h"
-#include "python/tensorstore/type_name_override.h"
+#include "python/tensorstore/type_name_override.h"  // IWYU pragma: keep
 #include "tensorstore/util/status.h"
 #include "tensorstore/util/str_cat.h"