Skip to content

Add skip_translate_if_provided option to bypass translation when target exists (multi-task mode only) - #7

Merged
trotacodigos merged 5 commits into
masterfrom
copilot/add-skip-translate-option
Dec 28, 2025
Merged

Add skip_translate_if_provided option to bypass translation when target exists (multi-task mode only)#7
trotacodigos merged 5 commits into
masterfrom
copilot/add-skip-translate-option

Conversation

Copilot AI commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

Implements optional translation skipping when input data already contains translations in the target column. This feature is only available for multi-task mode.

Changes

Multi-task mode (agents/modules/multitasker.py)

  • Skip translate step when skip_translate_if_provided: true and valid target exists
  • Pipeline proceeds directly to postedit → proofread

Configuration

  • Added skip_translate_if_provided: false to multi.yaml
  • Default false preserves existing behavior

Documentation

  • Updated README.md to clarify feature is multi-task mode only
  • Added complete example configuration for multi-task mode

Skip conditions (all required):

  • Config option enabled
  • target field exists, is string type, and non-empty

Usage

skip_translate_if_provided: true

model:
  translate:
    name: gpt-4.1-mini-2025-04-14
    temperature: 0.7
    max_tokens: 2048
  postedit:
    name: gpt-4o
    temperature: 0.7
    max_tokens: 1024
  proofread:
    name: gpt-5
    temperature: 0.7
    max_tokens: 1024

With this enabled in multi-task mode, rows containing translations in the target column bypass the translate agent entirely and proceed directly to postedit → proofread.

Original prompt

This section details on the original issue you should resolve

<issue_title>Feature request: skip translate agent when translation is already provided</issue_title>
<issue_description>The pipeline always invokes the translate agent, even when a translation already exists.

Proposed Feature

Add an option to skip the translate agent when a translation is already available.
This feature should be available 1) when translation is provided and 2) the task is not translate.</issue_description>

<agent_instructions>Keep the code clean and efficient while managing the issue.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits December 28, 2025 04:13
Co-authored-by: trotacodigos <82924238+trotacodigos@users.noreply.github.com>
Co-authored-by: trotacodigos <82924238+trotacodigos@users.noreply.github.com>
Co-authored-by: trotacodigos <82924238+trotacodigos@users.noreply.github.com>
Copilot AI changed the title [WIP] Add option to skip translate agent when translation exists Add skip_translate_if_provided option to bypass translation when target exists Dec 28, 2025
Copilot AI requested a review from trotacodigos December 28, 2025 04:18
@trotacodigos

Copy link
Copy Markdown
Owner

@copilot You're right that this feature should only be applied to multitasker, not on singletasker.

…r multi-task

Co-authored-by: trotacodigos <82924238+trotacodigos@users.noreply.github.com>

Copilot AI commented Dec 28, 2025

Copy link
Copy Markdown
Contributor Author

@copilot You're right that this feature should only be applied to multitasker, not on singletasker.

Removed the feature from single-task mode. The skip_translate_if_provided option now only applies to multitasker. Changes reverted in commit f812d0f.

Copilot AI changed the title Add skip_translate_if_provided option to bypass translation when target exists Add skip_translate_if_provided option to bypass translation when target exists (multi-task mode only) Dec 28, 2025
Comment thread agents/config/multi.yaml

# Skip translate agent if translation is already provided in the 'target' column
# Only applicable for multi-task mode
skip_translate_if_provided: false

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer the default value to be True.

@trotacodigos trotacodigos left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The default value of skip_translate_if_provided should be true.

@trotacodigos
trotacodigos marked this pull request as ready for review December 28, 2025 04:33
@trotacodigos
trotacodigos merged commit c121e06 into master Dec 28, 2025
1 check failed
Copilot AI requested a review from trotacodigos December 28, 2025 04:34
@trotacodigos
trotacodigos deleted the copilot/add-skip-translate-option branch December 28, 2025 04:34
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.

Feature request: skip translate agent when translation is already provided

2 participants