Handle two warnings that pollute the output of sssom-py CLI (#561)

See commits for a more detailed description of the changes;

This PR should not introduce any functional changes.

Author: matentzn

src/sssom/cli.py

@@ -54,6 +54,7 @@
     filter_redundant_rows,
     invert_mappings,
     merge_msdf,
+    pandas_set_no_silent_downcasting,
     reconcile_prefix_and_data,
     remove_unmatched,
     sort_df_rows_columns,
@@ -126,6 +127,9 @@
 def main(verbose: int, quiet: bool):
     """Run the SSSOM CLI."""
     logger = _logging.getLogger()
+
+    pandas_set_no_silent_downcasting()
+
     if verbose >= 2:
         logger.setLevel(level=_logging.DEBUG)
     elif verbose == 1:

src/sssom/parsers.py

@@ -424,9 +424,7 @@ def from_sssom_dataframe(
     # Need to revisit this solution.
     # This is to address: A value is trying to be set on a copy of a slice from a DataFrame
     if CONFIDENCE in df.columns:
-        df2 = df.copy()
-        df2[CONFIDENCE].replace(r"^\s*$", np.nan, regex=True, inplace=True)
-        df = df2
+        df.replace({CONFIDENCE: r"^\s*$"}, np.nan, regex=True, inplace=True)
 
     mapping_set = _get_mapping_set_from_df(df=df, meta=meta)
     doc = MappingSetDocument(mapping_set=mapping_set, converter=converter)

src/sssom/util.py

@@ -158,14 +158,9 @@ def from_mapping_set_document(cls, doc: MappingSetDocument) -> "MappingSetDataFr
         df = pd.DataFrame(get_dict_from_mapping(mapping) for mapping in doc.mapping_set.mappings)
         meta = _extract_global_metadata(doc)
 
-        if pandas_version >= (2, 0, 0):
-            # For pandas >= 2.0.0, use the 'copy' parameter
-            df = df.infer_objects(copy=False)
-        else:
-            # For pandas < 2.0.0, call 'infer_objects()' without any parameters
-            df = df.infer_objects()
         # remove columns where all values are blank.
         df.replace("", np.nan, inplace=True)
+        df = df.infer_objects()
         df.dropna(axis=1, how="all", inplace=True)  # remove columns with all row = 'None'-s.
 
         slots = _get_sssom_schema_object().dict["slots"]
@@ -1493,3 +1488,12 @@ def safe_compress(uri: str, converter: Converter) -> str:
     :return: A CURIE
     """
     return converter.compress_or_standardize(uri, strict=True)
+
+
+def pandas_set_no_silent_downcasting(no_silent_downcasting=True):
+    """Set pandas future.no_silent_downcasting option. Context https://github.com/pandas-dev/pandas/issues/57734."""
+    try:
+        pd.set_option("future.no_silent_downcasting", no_silent_downcasting)
+    except KeyError:
+        # Option does not exist in this version of pandas
+        pass