Skip to content

Change DEFAULT_LOCAL_DEV_MODEL_TYPE from "text-to-text" to "any-to-any" #680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jul 4, 2025

This PR addresses the FIXME comment in clarifai/utils/constants.py by changing the default local development model type from "text-to-text" to "any-to-any".

Changes Made

  1. clarifai/utils/constants.py: Changed DEFAULT_LOCAL_DEV_MODEL_TYPE from "text-to-text" to "any-to-any" and removed the FIXME comment
  2. clarifai/cli/model.py: Updated hardcoded default to use the constant instead of hardcoded "text-to-text"
  3. clarifai/cli/templates/model_templates.py: Updated the default parameter in get_config_template() from "text-to-text" to "any-to-any"

Impact

  • When users create new models via CLI without specifying a model type, they will now get "any-to-any" instead of "text-to-text"
  • The change is minimal and non-breaking - existing code that explicitly specifies model types will continue to work
  • Templates and configuration generation now consistently use "any-to-any" as the default

Before

DEFAULT_LOCAL_DEV_MODEL_TYPE = "text-to-text"  # FIXME comment indicated this should be any-to-any

After

DEFAULT_LOCAL_DEV_MODEL_TYPE = "any-to-any"

Fixes #679.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: sainivedh <39689508+sainivedh@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Use any-to-any as default Change DEFAULT_LOCAL_DEV_MODEL_TYPE from "text-to-text" to "any-to-any" Jul 4, 2025
@Copilot Copilot AI requested a review from sainivedh July 4, 2025 15:43
Copilot finished work on behalf of sainivedh July 4, 2025 15:43
Copy link

github-actions bot commented Jul 7, 2025

Code Coverage

Package Line Rate Health
clarifai 43%
clarifai.cli 45%
clarifai.cli.templates 28%
clarifai.client 69%
clarifai.client.auth 66%
clarifai.constants 100%
clarifai.datasets 100%
clarifai.datasets.export 80%
clarifai.datasets.upload 75%
clarifai.datasets.upload.loaders 37%
clarifai.models 100%
clarifai.modules 0%
clarifai.rag 72%
clarifai.runners 12%
clarifai.runners.models 58%
clarifai.runners.pipeline_steps 46%
clarifai.runners.pipelines 85%
clarifai.runners.utils 62%
clarifai.runners.utils.data_types 72%
clarifai.schema 100%
clarifai.urls 58%
clarifai.utils 70%
clarifai.utils.evaluation 67%
clarifai.workflows 95%
Summary 63% (7286 / 11477)

Minimum allowed line rate is 50%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Use any-to-any as default
2 participants