james-willis commented on code in PR #2167:
URL: https://github.com/apache/sedona/pull/2167#discussion_r2237589035
##########
python/tests/test_path_compatibility.py:
##########
@@ -118,3 +121,30 @@ def test_sql_module_imports(self):
def test_geoarrow_import(self):
# Test create_spatial_dataframe import
assert create_spatial_dataframe is not None
+ assert dataframe_to_arrow is not None
+
+ def test_raster_utils_imports(self):
+ # Test raster utils imports
+ assert SedonaUtils is not None
+
+ def test_import_df_functions_from_sedona_sql(self):
+ # one from each module
+ from sedona.sql import ST_MakePoint, ST_Y, ST_Touches, ST_Envelope_Aggr
+
+ def test_geoarrow_imports(self):
+ from sedona.geoarrow import create_spatial_dataframe,
dataframe_to_arrow
+
+ def test_sedona_util_imports(self):
+ from sedona.utils import KryoSerializer
+ from sedona.utils import SedonaKryoRegistrator
+
+ def test_maps_imports(self):
+ # Test Map imports
+ from sedona.maps import SedonaKepler, SedonaMapUtils, SedonaPyDeck
Review Comment:
I migrated the whole file to this scoped imports style. Are you ok with that
or do you want to convert it all to the same style as the
`test_path_compatability_all.py`file?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
