PR feedback

wandb/apis/public.py

@@ -1092,10 +1092,10 @@ def use_artifact(self, artifact):
             api.use_artifact(artifact.id)
             return artifact
         elif isinstance(artifact, wandb.Artifact):
-            raise ValueError("Only existing artifacts are excepted by this api. "
+            raise ValueError("Only existing artifacts are accepted by this api. "
                              "Manually create one with `wandb artifacts put`")
         else:
-            raise ValueError('You must wandb.Api().artifact() to use_artifact')
+            raise ValueError('You must pass a wandb.Api().artifact() to use_artifact')
 
     @normalize_exceptions
     def log_artifact(self, artifact, aliases=None):
@@ -1118,10 +1118,10 @@ def log_artifact(self, artifact, aliases=None):
                                 artifact.digest, aliases=aliases)
             return artifact
         elif isinstance(artifact, wandb.Artifact):
-            raise ValueError("Only existing artifacts are excepted by this api. "
+            raise ValueError("Only existing artifacts are accepted by this api. "
                              "Manually create one with `wandb artifacts put`")
         else:
-            raise ValueError('You must wandb.Api().artifact() to use_artifact')
+            raise ValueError('You must pass a wandb.Api().artifact() to use_artifact')
 
     @property
     def summary(self):

wandb/internal/sender.py

@@ -520,10 +520,11 @@ def send_artifact(self, data):
             is_user_created=artifact.user_created,
         )
 
+        metadata = json.loads(artifact.metadata) if artifact.metadata else None
         saver.save(
             type=artifact.type,
             name=artifact.name,
-            metadata=json.loads(artifact.metadata or "{}"),
+            metadata=metadata,
             description=artifact.description,
             aliases=artifact.aliases,
             use_after_commit=artifact.use_after_commit,