Version 2.3.1: correctly copy remote assets

Author: Douglas Blank

cometx/_version.py

@@ -12,5 +12,5 @@
 #  the express permission of Comet ML Inc.
 # *******************************************************
 
-version_info = (2, 3, 0)
+version_info = (2, 3, 1)
 __version__ = ".".join(map(str, version_info))

cometx/cli/copy.py

@@ -723,14 +723,26 @@ def log_assets(self, experiment, path, assets_metadata):
         for log_filename in assets_metadata:
             asset_type = assets_metadata[log_filename].get("type", "asset") or "asset"
             if asset_type not in ["confusion-matrix", "embeddings", "datagrid"]:
-                self._log_asset(
-                    experiment,
-                    path,
-                    asset_type,
-                    log_filename,
-                    assets_metadata,
-                    asset_map,
-                )
+                if (
+                    "remote" in assets_metadata[log_filename]
+                    and assets_metadata[log_filename]["remote"]
+                ):
+                    asset = assets_metadata[log_filename]
+                    experiment.log_remote_asset(
+                        uri=asset["link"],
+                        remote_file_name=asset["fileName"],
+                        step=asset["step"],
+                        metadata=asset["metadata"],
+                    )
+                else:
+                    self._log_asset(
+                        experiment,
+                        path,
+                        asset_type,
+                        log_filename,
+                        assets_metadata,
+                        asset_map,
+                    )
         # Process all nested assets:
         for log_filename in assets_metadata:
             asset_type = assets_metadata[log_filename].get("type", "asset") or "asset"

cometx/cli/copy_utils.py

@@ -15,7 +15,7 @@
 import logging
 import shutil
 
-from comet_ml.config_class import Config
+from comet_ml.config.config_class import Config
 from comet_ml.exceptions import (
     ExperimentAlreadyUploaded,
     ExperimentNotFound,

setup.py

@@ -46,7 +46,7 @@ def get_version(file, name="__version__"):
     long_description=long_description,
     long_description_content_type="text/markdown",
     install_requires=[
-        "comet_ml",
+        "comet_ml>=3.49.1",
     ],
     extras_require={
         "all": [