Performance improvement, allow functions to run in parallel rather than in the async event loop

It is because the underlying calls are synchronous by nature, in FastAPI endpoints marked with async are always ran in the eventloop even when blocking, when non async it attempts to coalasce and run requests in threads

Author: dominiquekleeven

src/service_ml_forecast/api/model_config_route.py

@@ -57,7 +57,7 @@
 )
 @realm_accessible
 @roles_allowed(resource=OPENREMOTE_KC_RESOURCE, roles=[ClientRoles.WRITE_ADMIN_ROLE])
-async def create_model_config(
+def create_model_config(
     user: Annotated[UserContext, Depends(KeycloakMiddleware.get_user_context)],
     realm: str,
     model_config: ModelConfig,
@@ -78,7 +78,7 @@ async def create_model_config(
 )
 @realm_accessible
 @roles_allowed(resource=OPENREMOTE_KC_RESOURCE, roles=[ClientRoles.READ_ADMIN_ROLE])
-async def get_model_config(
+def get_model_config(
     user: Annotated[UserContext, Depends(KeycloakMiddleware.get_user_context)],
     realm: str,
     id: UUID,
@@ -98,7 +98,7 @@ async def get_model_config(
 )
 @realm_accessible
 @roles_allowed(resource=OPENREMOTE_KC_RESOURCE, roles=[ClientRoles.READ_ADMIN_ROLE])
-async def get_model_configs(
+def get_model_configs(
     user: Annotated[UserContext, Depends(KeycloakMiddleware.get_user_context)],
     realm: str,
     config_service: ModelConfigService = Depends(get_config_service),
@@ -118,7 +118,7 @@ async def get_model_configs(
 )
 @realm_accessible
 @roles_allowed(resource=OPENREMOTE_KC_RESOURCE, roles=[ClientRoles.WRITE_ADMIN_ROLE])
-async def update_model_config(
+def update_model_config(
     user: Annotated[UserContext, Depends(KeycloakMiddleware.get_user_context)],
     realm: str,
     id: UUID,
@@ -143,7 +143,7 @@ async def update_model_config(
 )
 @realm_accessible
 @roles_allowed(resource=OPENREMOTE_KC_RESOURCE, roles=[ClientRoles.WRITE_ADMIN_ROLE])
-async def delete_model_config(
+def delete_model_config(
     user: Annotated[UserContext, Depends(KeycloakMiddleware.get_user_context)],
     realm: str,
     id: UUID,

src/service_ml_forecast/api/web_route.py

@@ -48,7 +48,7 @@
         HTTPStatus.NOT_FOUND: {"description": "Index.html file not found"},
     },
 )
-async def serve_index() -> FileResponse:
+def serve_index() -> FileResponse:
     """Serve the index.html file from the web dist directory."""
 
     index_path = DIRS.ML_WEBSERVER_UI_DIST_DIR / "index.html"
@@ -65,7 +65,7 @@ async def serve_index() -> FileResponse:
         HTTPStatus.NOT_FOUND: {"description": "Static file not found"},
     },
 )
-async def serve_spa(path: str) -> FileResponse:
+def serve_spa(path: str) -> FileResponse:
     """Serve static files or return index.html for SPA routing."""
 
     requested_path = DIRS.ML_WEBSERVER_UI_DIST_DIR / path