From d8c2d164455755168d328cf210b631a356d701be Mon Sep 17 00:00:00 2001 From: Heyuan Zeng Date: Mon, 6 Nov 2023 18:47:52 +0000 Subject: [PATCH] fix: typing hints --- src/gapper/core/result_synthesizer.py | 4 ++-- src/gapper/core/test_parameter.py | 12 ++++++++---- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/src/gapper/core/result_synthesizer.py b/src/gapper/core/result_synthesizer.py index 1a50963..b324502 100644 --- a/src/gapper/core/result_synthesizer.py +++ b/src/gapper/core/result_synthesizer.py @@ -60,8 +60,8 @@ class ResultSynthesizer: def __init__( self, *, - results: List[TestResult] = None, - post_tests: List[PostTest] = None, + results: List[TestResult] | None = None, + post_tests: List[PostTest] | None = None, metadata: GradescopeSubmissionMetadata | None = None, total_score: float | None = None, ) -> None: diff --git a/src/gapper/core/test_parameter.py b/src/gapper/core/test_parameter.py index a1211f3..75afed6 100644 --- a/src/gapper/core/test_parameter.py +++ b/src/gapper/core/test_parameter.py @@ -339,8 +339,6 @@ def __init__[ gap_description: str | Iterable[str] | Sequence[Iterable[str]] | None = None, gap_is_pipeline: bool | Sequence[bool] = False, gap_max_score: float | Sequence[float] | None = None, - gap_product: bool = False, - gap_zip: bool = False, gap_params: bool = False, gap_param_iter: bool = False, gap_singular_params: bool = False, @@ -371,8 +369,6 @@ def __init__[ gap_description: str | Iterable[str] | Sequence[Iterable[str]] | None = None, gap_is_pipeline: bool | Sequence[bool] = False, gap_weight: float | Sequence[float] | None = None, - gap_product: bool = False, - gap_zip: bool = False, gap_params: bool = False, gap_param_iter: bool = False, gap_singular_params: bool = False, @@ -396,12 +392,20 @@ def __init__( :param args: The arguments for the test parameter bundle. :param gap_product: Whether to take the cartesian product of the arguments. + .. deprecated:: + Use params, param_iter, singular_params, singular_param_iter instead. :param gap_zip: Whether to zip the arguments. + .. deprecated:: + Use params, param_iter, singular_params, singular_param_iter instead. :param gap_params: Whether to parse the arguments as parameters. :param gap_param_iter: Whether to parse the argument as parameter iterators. :param gap_singular_params: Whether to parse the arguments as singular parameters. :param gap_singular_param_iter: Whether to parse the arguments as singular parameter iterators. :param kwargs: The keyword arguments for the test parameter bundle. + .. seealso:: + :class:`gapper.core.test_parameter.TestParam` + seealso:: + :class:`gapper.core.test_parameter.ParamExtractor` """ if ( gap_params