Skip to content

Commit 61b659d

Browse files
authored
Merge pull request #1759 from roboflow/fix-modal-workspace-id-500
when runnning on modal we get 500 for workspace id params
2 parents d6f8b8f + 35d966c commit 61b659d

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

inference/core/interfaces/stream/inference_pipeline.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -596,10 +596,9 @@ def init_with_workflow(
596596
named_workflow_specified = (workspace_name is not None) and (
597597
workflow_id is not None
598598
)
599-
if not (named_workflow_specified != (workflow_specification is not None)):
599+
if not named_workflow_specified and not workflow_specification:
600600
raise ValueError(
601-
"Parameters (`workspace_name`, `workflow_id`) can be used mutually exclusive with "
602-
"`workflow_specification`, but at least one must be set."
601+
"Either (`workspace_name`, `workflow_id`) or `workflow_specification` must be provided."
603602
)
604603
try:
605604
from inference.core.interfaces.stream.model_handlers.workflows import (

inference/core/interfaces/webrtc_worker/modal.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,6 @@ def spawn_rtc_peer_connection_modal(
423423
workflow_id=webrtc_request.workflow_configuration.workflow_id,
424424
)
425425
)
426-
427426
tags = {"tag": docker_tag}
428427
if workspace_id:
429428
tags["workspace_id"] = workspace_id

0 commit comments

Comments
 (0)