Incorrect type hint for azure.ai.ml.sweep.Choice

Open Beginner friendly
#1,975 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
azure, python
Domain
machine-learning

Research direction

Search the Azure ML SDK definition of azure.ai.ml.sweep.Choice and inspect the values type hint. Replace the invariant list annotation with the requested sequence annotation, then verify that a list[str] value no longer produces a type error using the project's available type-checking or test workflow.

Written by the indexing model from the issue text.

Description

The definition of azure.ai.ml.sweep.Choice specifies values: List[float | str | dict] | None but the list type is invariant.
Normal usage would be to pass values: list[str], for example, which results in a type error.
List should be replaced with typing.Sequence in this context

Dominant language
Jupyter Notebook
Stars
4.4k
Forks
2.6k
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from Azure/MachineLearningNotebooks

All issues in Azure/MachineLearningNotebooks

Similar issues

More Machine Learning issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.