chore(pre_commit): ⬆ pre_commit autoupdate

.pre-commit-config.yaml

@@ -6,7 +6,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -24,14 +24,14 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/PyCQA/bandit
-    rev: 1.7.9
+    rev: 1.8.2
     hooks:
       - id: bandit
         args: ["-c", "pyproject.toml"]
         additional_dependencies: ["bandit[toml]"]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.6.4
+    rev: v0.9.1
     hooks:
       - id: ruff-format
       - id: ruff

roboflow/__init__.py

@@ -43,7 +43,7 @@ def check_key(api_key, model, notebook, num_retries=0):
                     num_retries += 1
                     return check_key(api_key, model, notebook, num_retries)
                 else:
-                    raise RuntimeError("There was an error validating the api key with Roboflow" " server.")
+                    raise RuntimeError("There was an error validating the api key with Roboflow server.")
             else:
                 r = response.json()
                 return r
@@ -71,7 +71,7 @@ def login(workspace=None, force=False):
     # default configuration location
     conf_location = os.getenv("ROBOFLOW_CONFIG_DIR", default=default_path)
     if os.path.isfile(conf_location) and not force:
-        write_line("You are already logged into Roboflow. To make a different login," "run roboflow.login(force=True).")
+        write_line("You are already logged into Roboflow. To make a different login,run roboflow.login(force=True).")
         return None
         # we could eventually return the workspace object here
         # return Roboflow().workspace()

roboflow/adapters/rfapi.py

@@ -179,7 +179,7 @@ def save_annotation(
 
 
 def _save_annotation_url(api_key, project_url, name, image_id, job_name, is_prediction, overwrite=False):
-    url = f"{API_URL}/dataset/{project_url}/annotate/{image_id}?api_key={api_key}" f"&name={name}"
+    url = f"{API_URL}/dataset/{project_url}/annotate/{image_id}?api_key={api_key}&name={name}"
     if job_name:
         url += f"&jobName={job_name}"
     if is_prediction:

roboflow/core/project.py

@@ -230,7 +230,7 @@ def generate_version(self, settings):
             )
 
         r = requests.post(
-            f"{API_URL}/{self.__workspace}/{self.__project_name}/" f"generate?api_key={self.__api_key}",
+            f"{API_URL}/{self.__workspace}/{self.__project_name}/generate?api_key={self.__api_key}",
             json=settings,
         )
 
@@ -426,7 +426,7 @@ def upload(
 
             if not is_image:
                 raise RuntimeError(
-                    "The image you provided {} is not a supported file format. We" " currently support: {}.".format(
+                    "The image you provided {} is not a supported file format. We currently support: {}.".format(
                         image_path, ", ".join(ACCEPTED_IMAGE_FORMATS)
                     )
                 )

roboflow/core/version.py

@@ -495,7 +495,7 @@ def deploy(self, model_type: str, model_path: str, filename: str = "weights/best
         ]
 
         if not any(supported_model in model_type for supported_model in supported_models):
-            raise (ValueError(f"Model type {model_type} not supported. Supported models are" f" {supported_models}"))
+            raise (ValueError(f"Model type {model_type} not supported. Supported models are {supported_models}"))
 
         if model_type.startswith(("paligemma", "paligemma2", "florence-2")):
             if any(model in model_type for model in ["paligemma", "paligemma2", "florence-2"]):
@@ -648,7 +648,7 @@ def deploy(self, model_type: str, model_path: str, filename: str = "weights/best
                     )
                 else:
                     if file in ["model_artifacts.json", "state_dict.pt"]:
-                        raise (ValueError(f"File {file} not found. Please make sure to provide a" " valid model path."))
+                        raise (ValueError(f"File {file} not found. Please make sure to provide a valid model path."))
 
         self.upload_zip(model_type, model_path)
 
@@ -761,7 +761,7 @@ def deploy_yolonas(self, model_type: str, model_path: str, filename: str = "weig
                     )
                 else:
                     if file in ["model_artifacts.json", filename]:
-                        raise (ValueError(f"File {file} not found. Please make sure to provide a" " valid model path."))
+                        raise (ValueError(f"File {file} not found. Please make sure to provide a valid model path."))
 
         self.upload_zip(model_type, model_path)
 
@@ -791,8 +791,7 @@ def upload_zip(self, model_type: str, model_path: str, model_file_name: str = "r
 
             if self.public:
                 print(
-                    "View the status of your deployment at:"
-                    f" {APP_URL}/{self.workspace}/{self.project}/{self.version}"
+                    f"View the status of your deployment at: {APP_URL}/{self.workspace}/{self.project}/{self.version}"
                 )
                 print(
                     "Share your model with the world at:"
@@ -801,8 +800,7 @@ def upload_zip(self, model_type: str, model_path: str, model_file_name: str = "r
                 )
             else:
                 print(
-                    "View the status of your deployment at:"
-                    f" {APP_URL}/{self.workspace}/{self.project}/{self.version}"
+                    f"View the status of your deployment at: {APP_URL}/{self.workspace}/{self.project}/{self.version}"
                 )
 
         except Exception as e:
@@ -824,7 +822,7 @@ def bar_progress(current, total, width=80):
             progress_message = (
                 "Downloading Dataset Version Zip in "
                 f"{location} to {format}: "
-                f"{current/total*100:.0f}% [{current} / {total}] bytes"
+                f"{current / total * 100:.0f}% [{current} / {total}] bytes"
             )
             sys.stdout.write("\r" + progress_message)
             sys.stdout.flush()
@@ -923,7 +921,7 @@ def __get_format_identifier(self, format):
 
         if not format:
             raise RuntimeError(
-                "You must pass a format argument to version.download() or define a" " model in your Roboflow object"
+                "You must pass a format argument to version.download() or define a model in your Roboflow object"
             )
 
         friendly_formats = {"yolov5": "yolov5pytorch", "yolov7": "yolov7pytorch"}

roboflow/core/workspace.py

@@ -269,7 +269,7 @@ def two_stage_ocr(
                 # capture OCR results from cropped image
                 results.append(ocr_infer(croppedImg)["results"])
         else:
-            print("please use an object detection model--can only perform two stage with" " bounding box results")
+            print("please use an object detection model--can only perform two stage with bounding box results")
 
         return results
 

roboflow/deployment.py

@@ -197,7 +197,7 @@ def get_deployment(args):
             print(json.dumps(msg, indent=2))
             break
 
-        print(f'{datetime.now().strftime("%H:%M:%S")} Waiting for deployment {args.deployment_name} to be ready...\n')
+        print(f"{datetime.now().strftime('%H:%M:%S')} Waiting for deployment {args.deployment_name} to be ready...\n")
         time.sleep(30)
 
 
@@ -278,7 +278,7 @@ def get_deployment_log(args):
                 continue
             log_ids.add(log["insert_id"])
             last_log_timestamp = log_timestamp
-            print(f'[{log_timestamp.strftime("%Y-%m-%d %H:%M:%S.%f")}] {log["payload"]}')
+            print(f"[{log_timestamp.strftime('%Y-%m-%d %H:%M:%S.%f')}] {log['payload']}")
 
         if not args.follow:
             break

roboflow/roboflowpy.py

@@ -249,7 +249,7 @@ def _add_upload_parser(subparsers):
     upload_parser.add_argument(
         "-w",
         dest="workspace",
-        help="specify a workspace url or id " "(will use default workspace if not specified)",
+        help="specify a workspace url or id (will use default workspace if not specified)",
     )
     upload_parser.add_argument(
         "-p",
@@ -307,7 +307,7 @@ def _add_import_parser(subparsers):
     import_parser.add_argument(
         "-w",
         dest="workspace",
-        help="specify a workspace url or id " "(will use default workspace if not specified)",
+        help="specify a workspace url or id (will use default workspace if not specified)",
     )
     import_parser.add_argument(
         "-p",

roboflow/util/versions.py

@@ -34,7 +34,7 @@ def get_wrong_dependencies_versions(
         module = import_module(dependency)
         module_version = module.__version__
         if order not in order_funcs:
-            raise ValueError(f"order={order} not supported, please use" f" `{', '.join(order_funcs.keys())}`")
+            raise ValueError(f"order={order} not supported, please use `{', '.join(order_funcs.keys())}`")
 
         is_okay = order_funcs[order](Version(module_version), Version(version))
         if not is_okay:
@@ -53,7 +53,7 @@ def print_warn_for_wrong_dependencies_versions(
             f" {dependency}{order}{version}`"
         )
         if ask_to_continue:
-            answer = input(f"Would you like to continue with the wrong version of {dependency}?" " y/n: ")
+            answer = input(f"Would you like to continue with the wrong version of {dependency}? y/n: ")
             if answer.lower() != "y":
                 sys.exit(1)
 

tests/test_project.py

@@ -30,7 +30,7 @@ def test_check_valid_image_with_unaccepted_formats(self):
     def test_upload_raises_upload_image_error(self):
         responses.add(
             responses.POST,
-            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}" f"&batch={DEFAULT_BATCH_NAME}",
+            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}&batch={DEFAULT_BATCH_NAME}",
             json={
                 "error": {
                     "message": "Invalid image.",
@@ -56,15 +56,15 @@ def test_upload_raises_upload_annotation_error(self):
         # Image upload
         responses.add(
             responses.POST,
-            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}" f"&batch={DEFAULT_BATCH_NAME}",
+            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}&batch={DEFAULT_BATCH_NAME}",
             json={"success": True, "id": image_id},
             status=200,
         )
 
         # Annotation
         responses.add(
             responses.POST,
-            f"{API_URL}/dataset/{PROJECT_NAME}/annotate/{image_id}?api_key={ROBOFLOW_API_KEY}" f"&name={image_name}",
+            f"{API_URL}/dataset/{PROJECT_NAME}/annotate/{image_id}?api_key={ROBOFLOW_API_KEY}&name={image_name}",
             json={
                 "error": {
                     "message": "Image was already annotated.",

tests/test_queries.py

@@ -47,7 +47,7 @@ def test_project_methods(self):
         # Upload image
         responses.add(
             responses.POST,
-            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}" f"&batch={DEFAULT_BATCH_NAME}",
+            f"{API_URL}/dataset/{PROJECT_NAME}/upload?api_key={ROBOFLOW_API_KEY}&batch={DEFAULT_BATCH_NAME}",
             json={"duplicate": True, "id": "hbALkCFdNr9rssgOUXug"},
             status=200,
         )