remove dev features completely from client. was done in e2e tests in #206

Author: bisgaard-itis

clients/python/src/osparc/_api_files_api.py

@@ -33,7 +33,6 @@
     DEFAULT_TIMEOUT_SECONDS,
     PaginationGenerator,
     compute_sha256,
-    dev_features_enabled,
     file_chunk_generator,
 )
 
@@ -60,11 +59,6 @@ def __init__(self, api_client: ApiClient):
             else None
         )
 
-    def __getattr__(self, name: str) -> Any:
-        if (name in FilesApi._dev_features) and (not dev_features_enabled()):
-            raise NotImplementedError(f"FilesApi.{name} is still under development")
-        return super().__getattribute__(name)
-
     def download_file(
         self,
         file_id: str,

clients/python/src/osparc/_api_solvers_api.py

@@ -1,6 +1,6 @@
 # Wraps osparc_client.api.solvers_api
 
-from typing import Any, List, Optional
+from typing import List, Optional
 
 import httpx
 from osparc_client.api.solvers_api import SolversApi as _SolversApi
@@ -12,8 +12,6 @@
     _DEFAULT_PAGINATION_LIMIT,
     _DEFAULT_PAGINATION_OFFSET,
     PaginationGenerator,
-    dev_feature,
-    dev_features_enabled,
 )
 
 import warnings
@@ -41,11 +39,6 @@ def __init__(self, api_client: ApiClient):
             else None
         )
 
-    def __getattr__(self, name: str) -> Any:
-        if (name in SolversApi._dev_features) and (not dev_features_enabled()):
-            raise NotImplementedError(f"SolversApi.{name} is still under development")
-        return super().__getattribute__(name)
-
     def list_solver_ports(
         self, solver_key: str, version: str, **kwargs
     ) -> List[SolverPort]:
@@ -54,7 +47,6 @@ def list_solver_ports(
         )
         return page.items if page.items else []
 
-    @dev_feature
     def iter_jobs(self, solver_key: str, version: str, **kwargs) -> PaginationGenerator:
         """Returns an iterator through which one can iterate over
         all Jobs submitted to the solver
@@ -87,7 +79,6 @@ def _pagination_method():
             auth=self._auth,
         )
 
-    @dev_feature
     def jobs(self, solver_key: str, version: str, **kwargs) -> PaginationGenerator:
         warnings.warn(
             "The 'jobs' method is deprecated and will be removed in a future version. "

clients/python/src/osparc/_api_studies_api.py

@@ -4,7 +4,7 @@
 import logging
 from pathlib import Path
 from tempfile import mkdtemp
-from typing import Any, Optional
+from typing import Optional
 
 import httpx
 from .models import JobInputs, JobLogsMap, PageStudy
@@ -18,7 +18,6 @@
     _DEFAULT_PAGINATION_LIMIT,
     _DEFAULT_PAGINATION_OFFSET,
     PaginationGenerator,
-    dev_features_enabled,
 )
 import warnings
 
@@ -58,11 +57,6 @@ def __init__(self, api_client: ApiClient):
             else None
         )
 
-    def __getattr__(self, name: str) -> Any:
-        if (name in StudiesApi._dev_features) and (not dev_features_enabled()):
-            raise NotImplementedError(f"StudiesApi.{name} is still under development")
-        return super().__getattribute__(name)
-
     def create_study_job(self, study_id: str, job_inputs: JobInputs, **kwargs):
         kwargs = {**kwargs, **ParentProjectInfo().model_dump(exclude_none=True)}
         return super().create_study_job(study_id, job_inputs, **kwargs)

clients/python/src/osparc/_utils.py

@@ -1,7 +1,6 @@
 import asyncio
 import hashlib
 import os
-from functools import wraps
 from pathlib import Path
 from typing import AsyncGenerator, Callable, Generator, Optional, Tuple, TypeVar, Union
 
@@ -123,13 +122,3 @@ async def compute_sha256(file: Path) -> str:
 
 def dev_features_enabled() -> bool:
     return os.environ.get("OSPARC_DEV_FEATURES_ENABLED") == "1"
-
-
-def dev_feature(func: Callable):
-    @wraps(func)
-    def _wrapper(*args, **kwargs):
-        if not dev_features_enabled():
-            raise NotImplementedError(f"{func.__name__} is still under development")
-        return func(*args, **kwargs)
-
-    return _wrapper