Fix various typos (#323)

Author: folmos-at-orange

.github/workflows/unit-tests.yml

@@ -214,7 +214,7 @@ jobs:
           git config --global --add safe.directory $(Resolve-Path '.' | % {$_.toString()})
           python setup.py egg_info
 
-          # Install the Pyrhon requirements
+          # Install the Python requirements
           Get-Content .\khiops.egg-info\requires.txt `
             | Select-String -Pattern '^\[' -NotMatch `
             | Select-String -Pattern '^$' -NotMatch `

CHANGELOG.md

@@ -87,7 +87,7 @@
 
 ### Fixed
 - `core`
-  - Metric name search in estimator analyis report.
+  - Metric name search in estimator analysis report.
 
 ## 10.2.1.0 - 2024-03-26
 
@@ -219,7 +219,7 @@ Note: This release marks the open sourcing of Khiops:
   - Estimators now accept dataframes with numerical column indexes.
   - `KhiopsClassifier` now accepts integer target vectors.
   - `classes_` estimator attribute for `KhiopsClassifier` (available once fitted).
-  - `feature_names_out_` estimator attirbute for `KhiopsEncoder` (available once fitted).
+  - `feature_names_out_` estimator attribute for `KhiopsEncoder` (available once fitted).
   - `export_report_file` and `export_dictionary_file` to export Khiops report and dictionary files
     once the estimators are fitted.
   - `internal_sort` parameter for estimators that may be used to not sort the tables on the

doc/README.md

@@ -15,7 +15,7 @@ Khiops Python library.
 
 # You'll also need a system-wide installation of pandoc (https://pandoc.org)
 
-# Execute this if there were non commited updates to samples.py or samples_sklearn.py:
+# Execute this if there were non committed updates to samples.py or samples_sklearn.py:
 # ./convert-samples-hook
 
 # To clean the html documentation

doc/samples/samples_sklearn.rst

@@ -400,7 +400,7 @@ Samples
         os.path.join(accidents_dataset_path, "Vehicles.txt"), sep="\t"
     )
 
-    # Split the secondary dataframe with the keys of the splitted root dataframe
+    # Split the secondary dataframe with the keys of the split root dataframe
     X_train_ids = X_train_main["AccidentId"].to_frame()
     X_test_ids = X_test_main["AccidentId"].to_frame()
     X_train_secondary = X_train_ids.merge(vehicles_df, on="AccidentId")
@@ -825,7 +825,7 @@ Samples
     # Load the secondary table of the dataset into a pandas dataframe
     vehicles_df = pd.read_csv(os.path.join(accidents_data_dir, "Vehicles.txt"), sep="\t")
 
-    # Split the secondary dataframe with the keys of the splitted root dataframe
+    # Split the secondary dataframe with the keys of the split root dataframe
     X_train_ids = X_train["AccidentId"].to_frame()
     X_test_ids = X_test["AccidentId"].to_frame()
     X_train_secondary = X_train_ids.merge(vehicles_df, on="AccidentId")
@@ -891,7 +891,7 @@ Samples
         os.path.join(accidents_dataset_path, "Vehicles.txt"), sep="\t"
     )
 
-    # Split the secondary dataframe with the keys of the splitted root dataframe
+    # Split the secondary dataframe with the keys of the split root dataframe
     X_train_ids = X_train_main["AccidentId"].to_frame()
     X_test_ids = X_test_main["AccidentId"].to_frame()
     X_train_secondary = X_train_ids.merge(vehicles_df, on="AccidentId")

khiops/core/api.py

@@ -757,7 +757,7 @@ def train_predictor(
         Pairs specified with ``specific_pairs`` have top priority: they are constructed
         first.
     only_pairs_with : str, default ""
-        Constructs only pairs with the specifed variable name. If equal to the empty
+        Constructs only pairs with the specified variable name. If equal to the empty
         string "" it considers all variables to make pairs.
         **Deprecated** will be removed in Khiops Python 11, use ``specific_pairs``.
     group_target_value : bool, default ``False``
@@ -1083,7 +1083,7 @@ def train_recoder(
         Pairs specified with ``specific_pairs`` have top priority: they are constructed
         first.
     only_pairs_with : str, default ""
-        Constructs only pairs with the specifed variable name. If equal to the empty
+        Constructs only pairs with the specified variable name. If equal to the empty
         string "" it considers all variables to make pairs.
         **Deprecated** will be removed in Khiops Python 11, use ``specific_pairs``.
     group_target_value : bool, default ``False``
@@ -1133,7 +1133,7 @@ def train_recoder(
             - "conditional info": Conditional information of the interval/group
             - "none": Keeps the variable as-is
     grouping_method : str
-        Name of the grouping method. Its vaild values depend on the task:
+        Name of the grouping method. Its valid values depend on the task:
             - Supervised: "MODL" (default) or "BasicGrouping"
             - Unsupervised: "BasicGrouping" (default) or "None"
     min_group_frequency : int, default 0

khiops/core/coclustering_results.py

@@ -490,7 +490,7 @@ class CoclusteringDimension:
     """A coclustering dimension (variable)
 
     A coclustering dimension is a hierarchical clustering of an input variable. The
-    leafs of this hierarchy are linked to an element of a partition of the input
+    leaves of this hierarchy are linked to an element of a partition of the input
     variable. Leaf clusters have variable parts as their children.
 
     It only has a no-parameter constructor.
@@ -553,7 +553,7 @@ def __init__(self):
         # Default group attribute only for categorical dimensions
         self.default_group = None
 
-        # List of the hierarchy clusters, ranging from the root to the leafs
+        # List of the hierarchy clusters, ranging from the root to the leaves
         self.root_cluster = None
         self.clusters = []
 
@@ -939,7 +939,7 @@ def __init__(self, json_data=None):
         # Transform to an empty dictionary if json_data is not specified
         if json_data is None:
             json_data = {}
-        # Otherise check the validity of json_data
+        # Otherwise check the validity of json_data
         elif "cluster" not in json_data:
             raise KhiopsJSONError("'cluster' key not found")
 
@@ -955,7 +955,7 @@ class CoclusteringDimensionPartInterval(CoclusteringDimensionPart):
     json_data : dict, optional
         Python dictionary representing an element of type "Numerical" of the list at the
         ``dimensionPartitions`` field of a Khiops Coclustering JSON report file. If not
-        specifed it returns an empty instance.
+        specified it returns an empty instance.
 
     Raises
     ------
@@ -1205,7 +1205,7 @@ def __init__(self, json_data=None):
         # Transform to an empty dictionary if json_data is not specified
         if json_data is None:
             json_data = {}
-        # Otherwise check the "cluter" and "parentCluster" keys
+        # Otherwise check the "cluster" and "parentCluster" keys
         else:
             if "cluster" not in json_data:
                 raise KhiopsJSONError("'cluster' key not found")
@@ -1224,7 +1224,7 @@ def __init__(self, json_data=None):
         self.short_description = json_data.get("shortDescription", "")
         self.description = json_data.get("description", "")
 
-        # Link to child clusters, None for the leafs of the hierarchy
+        # Link to child clusters, None for the leaves of the hierarchy
         # The user must specify the CoclusteringCluster references parent_cluster
         # and child_cluster that link this instance to the hierarchy
         self.parent_cluster = None

khiops/core/internals/filesystems.py

@@ -19,7 +19,7 @@
 # pylint: disable=invalid-name
 
 # Import boto3 if available
-# Delay the raising of an ImportError to an instantation of a AmazonS3Resource
+# Delay an ImportError raising to an instantiation of a AmazonS3Resource
 try:
     import boto3
     import boto3.session
@@ -30,7 +30,7 @@
     boto3_import_error = import_error
 
 # Import google.could if available
-# Delay the raising of an ImportError to an instantation of a GoogleCloudStorageResource
+# Delay an ImportError raising to an instantiation of a GoogleCloudStorageResource
 try:
     from google.cloud import storage
 

khiops/core/internals/runner.py

@@ -365,7 +365,7 @@ def samples_dir(self):
         return self._get_samples_dir()
 
     def _get_samples_dir(self):
-        """To be overriden by subclasses"""
+        """To be overridden by subclasses"""
         return self._samples_dir
 
     @samples_dir.setter
@@ -386,7 +386,7 @@ def khiops_version(self):
         return self._get_khiops_version()
 
     def _get_khiops_version(self):
-        """khiops_version getter to be overriden by subclasses"""
+        """khiops_version getter to be overridden by subclasses"""
         return self._khiops_version
 
     def _build_status_message(self):
@@ -1072,7 +1072,7 @@ def _set_samples_dir(self, samples_dir):
         super()._set_samples_dir(samples_dir)
 
     def _get_samples_dir(self):
-        # Check the samples dir once (the check emmits only warnings)
+        # Check the samples dir once (the check emits only warnings)
         if not self._samples_dir_checked:
             _check_samples_dir(self._samples_dir)
             self._samples_dir_checked = True

khiops/core/internals/scenario.py

@@ -82,7 +82,7 @@ def _parse_template(self):
             line = next(line_iter, None)
             line_num += 1
 
-        # Eliminate empty lines at the beggining and end if any
+        # Eliminate empty lines at the beginning and end if any
         if self._parsed_template[0] == "":
             self._parsed_template.pop(0)
         if self._parsed_template[-1] == "":
@@ -128,7 +128,7 @@ def _parse_section(self, section_keyword, line_iter):
             if statement_re.match(statement) is None:
                 raise ValueError(
                     "Statement must contain only alphabetic characters and '.' "
-                    f"(no '.' at the beggining): '{statement}'"
+                    f"(no '.' at the beginning): '{statement}'"
                 )
 
         return section_param_name, section_spec

khiops/core/internals/task.py

@@ -210,7 +210,7 @@ def __init__(
             for kwarg_signature in self.kwargs_signature
         }
 
-        # Post-intialization checks
+        # Post-initialization checks
         # Check that the path_valued_arg_names are contained in either args or kwargs
         all_arg_names = list(self._args_signature_by_name.keys()) + list(
             self._kwargs_signature_by_name.keys()