Repository: great-expectations/great_expectations

Issue:
Use cleaner solution for non-truncating division in python 2
Prefer `from __future__ import division` to `1.*x/y`


Discussion:
Done inline during improve_partitions push.

Resolving patch:
diff --git a/docs/sphinx_api_docs_source/public_api_report.py b/docs/sphinx_api_docs_source/public_api_report.py
--- a/docs/sphinx_api_docs_source/public_api_report.py
+++ b/docs/sphinx_api_docs_source/public_api_report.py
@@ -834,14 +834,20 @@ class CodeReferenceFilter:
             ),
         ),
         IncludeExcludeDefinition(
-            reason="Experimental is not part of the public API",
-            filepath=pathlib.Path(
-                "great_expectations/experimental/datasources/interfaces.py"
-            ),
+            reason="Fluent is not part of the public API",
+            filepath=pathlib.Path("great_expectations/datasource/fluent/interfaces.py"),
         ),
         IncludeExcludeDefinition(
-            reason="Experimental is not part of the public API",
-            filepath=pathlib.Path("great_expectations/experimental/context.py"),
+            reason="Fluent is not part of the public API",
+            name="read_csv",
+            filepath=pathlib.Path("great_expectations/datasource/fluent/config.py"),
+        ),
+        IncludeExcludeDefinition(
+            reason="Fluent-style read_csv is not referenced in the docs yet, but due to string matching it is being flagged.",
+            name="read_csv",
+            filepath=pathlib.Path(
+                "great_expectations/datasource/fluent/pandas_datasource.py"
+            ),
         ),
         IncludeExcludeDefinition(
             reason="Marshmallow dump methods are not part of the public API",
@@ -1471,13 +1477,6 @@ class CodeReferenceFilter:
             name="get_available_data_asset_names",
             filepath=pathlib.Path("great_expectations/datasource/datasource.py"),
         ),
-        IncludeExcludeDefinition(
-            reason="Fluent-style read_csv is not referenced in the docs yet, but due to string matching it is being flagged.",
-            name="read_csv",
-            filepath=pathlib.Path(
-                "great_expectations/experimental/datasources/pandas_datasource.py"
-            ),
-        ),
     ]
 
     def __init__(
@@ -1844,7 +1843,7 @@ def main():
     # any methods or classes you are adding to documentation with the @public_api
     # decorator and any relevant "new" or "deprecated" public api decorators.
     # If the actual is lower than the threshold, please reduce the threshold.
-    PUBLIC_API_MISSING_THRESHOLD = 100  # TODO: reduce this number again once this works for the Fluent DS dynamic methods
+    PUBLIC_API_MISSING_THRESHOLD = 97  # TODO: reduce this number again once this works for the Fluent DS dynamic methods
     if len(printable_definitions) != PUBLIC_API_MISSING_THRESHOLD:
         error_msg_prefix = f"There are {len(printable_definitions)} items missing from the public API, we currently allow {PUBLIC_API_MISSING_THRESHOLD}."
         if len(printable_definitions) > PUBLIC_API_MISSING_THRESHOLD:
diff --git a/great_expectations/datasource/fluent/config.py b/great_expectations/datasource/fluent/config.py
--- a/great_expectations/datasource/fluent/config.py
+++ b/great_expectations/datasource/fluent/config.py
@@ -24,7 +24,6 @@
 from ruamel.yaml import YAML
 from typing_extensions import Final
 
-from great_expectations.core._docs_decorators import public_api
 from great_expectations.datasource.fluent.fluent_base_model import (
     FluentBaseModel,
     _recursively_set_config_value,
@@ -212,7 +211,6 @@ def yaml(
     ) -> pathlib.Path:
         ...
 
-    @public_api
     def yaml(
         self,
         stream_or_path: Union[StringIO, pathlib.Path, None] = None,
@@ -255,7 +253,6 @@ def yaml(
 
         return stream_or_path.getvalue()
 
-    @public_api
     def json(
         self,
         *,
@@ -328,7 +325,6 @@ def _json_dict(
             )
         )
 
-    @public_api
     def dict(
         self,
         *,
diff --git a/great_expectations/datasource/fluent/fluent_base_model.py b/great_expectations/datasource/fluent/fluent_base_model.py
--- a/great_expectations/datasource/fluent/fluent_base_model.py
+++ b/great_expectations/datasource/fluent/fluent_base_model.py
@@ -22,7 +22,6 @@
 import pydantic
 from ruamel.yaml import YAML
 
-from great_expectations.core._docs_decorators import public_api
 from great_expectations.datasource.fluent.config_str import ConfigStr
 from great_expectations.datasource.fluent.constants import _FIELDS_ALWAYS_SET
 
@@ -102,7 +101,6 @@ def yaml(
     ) -> pathlib.Path:
         ...
 
-    @public_api
     def yaml(
         self,
         stream_or_path: Union[StringIO, pathlib.Path, None] = None,
@@ -144,7 +142,6 @@ def yaml(
             return stream_or_path
         return stream_or_path.getvalue()
 
-    @public_api
     def json(
         self,
         *,
@@ -216,7 +213,6 @@ def _json_dict(
             )
         )
 
-    @public_api
     def dict(
         self,
         *,


Test patch: