Skip to content
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

Deprecate and remove processor_ids argument by v1.4 #6271

Closed
jurruti opened this issue Aug 30, 2023 · 4 comments
Closed

Deprecate and remove processor_ids argument by v1.4 #6271

jurruti opened this issue Aug 30, 2023 · 4 comments
Assignees
Labels
good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. kind/health For CI/testing/release process/refactoring/technical debt items triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on

Comments

@jurruti
Copy link
Contributor

jurruti commented Aug 30, 2023

The run methods in Engine, EngineProgram and EngineClient now include a parameter called processor_id. This parameter can be used to specify a specific processor for the run, and will substitute the deprecated parameter processor_ids , because allowing multiple candidate processors was not useful in practice.

@jurruti jurruti added the kind/health For CI/testing/release process/refactoring/technical debt items label Aug 30, 2023
@SudhanshuJoshi09
Copy link

Hey @jurruti , could I pick this up ?

@verult verult self-assigned this Jan 3, 2024
@verult verult added the triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on label Jan 3, 2024
@verult
Copy link
Collaborator

verult commented Jan 4, 2024

Hi @SudhanshuJoshi09 , are you still interested in taking this issue?

@dstrain115 dstrain115 added the good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. label Feb 13, 2024
@JaShom
Copy link
Contributor

JaShom commented Apr 1, 2024

Could I take on this issue please @verult ?

@pavoljuhas
Copy link
Collaborator

Could I take on this issue please @verult ?

Sure @JaShom, thank you for looking into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. kind/health For CI/testing/release process/refactoring/technical debt items triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on
Projects
None yet
Development

No branches or pull requests

6 participants