Skip to content

Allow loading locally stored models in workflows#2479

Merged
grzegorz-roboflow merged 6 commits into
mainfrom
feat/allow-local-models-in-workflows
Jun 18, 2026
Merged

Allow loading locally stored models in workflows#2479
grzegorz-roboflow merged 6 commits into
mainfrom
feat/allow-local-models-in-workflows

Conversation

@grzegorz-roboflow

Copy link
Copy Markdown
Collaborator

What does this PR do?

When ALLOW_INFERENCE_MODELS_DIRECTLY_ACCESS_LOCAL_PACKAGES is set, model ID like "/path/to/model/dir" should be accepted. Currently model manager tries to check the model type by making call to Roboflow API resulting in error (since "/path/to/model/dir" is not correct Roboflow model ID). This change fixes this behavior so locally stored models can be loaded and used with workflows.

Type of Change

  • New feature (non-breaking change that adds functionality)

Testing

  • I have tested this change locally

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where necessary, particularly in hard-to-understand areas
  • My changes generate no new warnings or errors
  • I have updated the documentation accordingly (if applicable)

Additional Context

N/A

Comment thread inference/core/registries/roboflow.py Outdated
@grzegorz-roboflow grzegorz-roboflow marked this pull request as draft June 18, 2026 11:13
@grzegorz-roboflow grzegorz-roboflow marked this pull request as ready for review June 18, 2026 16:13
@grzegorz-roboflow grzegorz-roboflow merged commit 6b3fc53 into main Jun 18, 2026
49 of 55 checks passed
@grzegorz-roboflow grzegorz-roboflow deleted the feat/allow-local-models-in-workflows branch June 18, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants