diff --git a/python/cuopt_self_hosted/cuopt_sh_client/thin_client_solver_settings.py b/python/cuopt_self_hosted/cuopt_sh_client/thin_client_solver_settings.py index 96e2e8cb9..9d2139931 100644 --- a/python/cuopt_self_hosted/cuopt_sh_client/thin_client_solver_settings.py +++ b/python/cuopt_self_hosted/cuopt_sh_client/thin_client_solver_settings.py @@ -167,7 +167,7 @@ def toDict(self): for key in self.parameter_dict: if "tolerance" not in key: solver_config[key] = self.parameter_dict[key] - # Handle tolerance seperately + # Handle tolerance separately if "absolute_dual_tolerance" in self.parameter_dict: solver_config["tolerances"]["absolute_dual"] = self.parameter_dict[ "absolute_dual_tolerance" diff --git a/python/cuopt_server/cuopt_server/utils/job_queue.py b/python/cuopt_server/cuopt_server/utils/job_queue.py index 31fed3912..7d1986e56 100644 --- a/python/cuopt_server/cuopt_server/utils/job_queue.py +++ b/python/cuopt_server/cuopt_server/utils/job_queue.py @@ -1202,7 +1202,7 @@ def __init__( self.warmstart_data = warmstart_data def delete_data(self): - # This is for cases where we skip a job on cancelation + # This is for cases where we skip a job on cancellation # In this case for shared memory use, we need # to unlink the shared memory if it is not a cache # reference because we will never call _resolve_job diff --git a/python/cuopt_server/cuopt_server/webserver.py b/python/cuopt_server/cuopt_server/webserver.py index 6c84445d2..edbc8234d 100644 --- a/python/cuopt_server/cuopt_server/webserver.py +++ b/python/cuopt_server/cuopt_server/webserver.py @@ -347,7 +347,8 @@ def getsolverlogs( detail=f"Unsupported Accept value {accept}, " f"supported values are {[mime_json, mime_msgpack, mime_zlib]}", ) - # TODO: Create a log dir + + # result_dir is guaranteed to exist on startup log_dir, _, _ = settings.get_result_dir() log_fname = "log_" + id log_file = os.path.join(log_dir, log_fname) @@ -637,7 +638,7 @@ def deletesolution( @app.delete( "/cuopt/request/{id}", - description="Note: for use with self hosted cuOpt instances. " + description="Note: for use with self-hosted cuOpt instances. " "Delete a request (either a cached request or a request to be solved). " "The 'id' is the reqId value returned from a POST to /cuopt/request.", summary="Delete a request by id (self-hosted)", @@ -1095,7 +1096,7 @@ async def postrequest( r.unregister_result() raise HTTPException( status_code=422, - detail=f"Requtest id '{reqId}' does not exist", + detail=f"Request id '{reqId}' does not exist", ) # if reqId is not set and we're not using file_path, stream in the data @@ -1243,7 +1244,7 @@ async def get_body(request: Request): responses=ManagedRequestResponse, # include_in_schema=False, # This form is necessary to allow multiple literal examples to - # be added to the Swaggger and redoc UIs. Noe the schema is + # be added to the Swagger and redoc UIs. Noe the schema is # taken by fastapi from the cuoptData parameter. openapi_extra={ "requestBody": {