Skip to content

[tractor] Override submit method from BaseSubmitter#4

Merged
cbentejac merged 1 commit intomasterfrom
temp/freeze_current_submitting_process
Apr 28, 2026
Merged

[tractor] Override submit method from BaseSubmitter#4
cbentejac merged 1 commit intomasterfrom
temp/freeze_current_submitting_process

Conversation

@Alxiice
Copy link
Copy Markdown
Collaborator

@Alxiice Alxiice commented Feb 19, 2026

Description

In alicevision/Meshroom#2984 we change the submitting process.
To avoid breaking the submitting in production, we propose :

  • first merge this PR
  • then merge Meshroom/2984 : the submitted scenes will not create pre/post tasks but will work like they used to do before
  • then merge the PR that will update the submitting process to be able to use pre/post tasks

@Alxiice Alxiice self-assigned this Feb 19, 2026
@Alxiice Alxiice force-pushed the temp/freeze_current_submitting_process branch from b609f43 to fd1e4fa Compare March 5, 2026 11:09
@Alxiice Alxiice force-pushed the temp/freeze_current_submitting_process branch from fd1e4fa to 1461077 Compare April 21, 2026 09:40
@cbentejac cbentejac force-pushed the temp/freeze_current_submitting_process branch from 1461077 to 5a79421 Compare April 28, 2026 14:40
@cbentejac cbentejac changed the title Temporary correction to make sure we don't break the submitting process [tractor] Override submit method of BaseSubmitter Apr 28, 2026
@cbentejac cbentejac changed the title [tractor] Override submit method of BaseSubmitter [tractor] Override submit method for BaseSubmitter Apr 28, 2026
@cbentejac cbentejac changed the title [tractor] Override submit method for BaseSubmitter [tractor] Override submit method from BaseSubmitter Apr 28, 2026
@cbentejac cbentejac merged commit cd9eefa into master Apr 28, 2026
@cbentejac cbentejac deleted the temp/freeze_current_submitting_process branch April 28, 2026 14:59
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