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

Swap is_multiseries logic to problem type #4278

Merged
merged 4 commits into from
Aug 17, 2023

Conversation

eccabay
Copy link
Contributor

@eccabay eccabay commented Aug 16, 2023

No description provided.

@machineFL
Copy link
Collaborator

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@codecov
Copy link

codecov bot commented Aug 16, 2023

Codecov Report

Merging #4278 (0a0edbb) into 4266_msts_search (9f3cb79) will decrease coverage by 15.0%.
The diff coverage is 65.4%.

@@                 Coverage Diff                 @@
##           4266_msts_search   #4278      +/-   ##
===================================================
- Coverage              99.7%   84.7%   -15.0%     
===================================================
  Files                   355     355              
  Lines                 39067   39065       -2     
===================================================
- Hits                  38946   33050    -5896     
- Misses                  121    6015    +5894     
Files Changed Coverage Δ
...lml/automl/automl_algorithm/iterative_algorithm.py 80.9% <ø> (-16.5%) ⬇️
evalml/automl/utils.py 81.1% <ø> (-16.2%) ⬇️
evalml/pipelines/components/component_base.py 100.0% <ø> (ø)
...sors/multiseries_time_series_baseline_regressor.py 100.0% <ø> (ø)
evalml/problem_types/__init__.py 100.0% <ø> (ø)
evalml/tests/automl_tests/test_automl.py 0.0% <0.0%> (-99.4%) ⬇️
evalml/tests/automl_tests/test_automl_utils.py 0.0% <0.0%> (-100.0%) ⬇️
...valml/tests/automl_tests/test_default_algorithm.py 0.0% <0.0%> (-100.0%) ⬇️
...lml/tests/automl_tests/test_iterative_algorithm.py 0.0% <0.0%> (-100.0%) ⬇️
evalml/tests/component_tests/test_utils.py 99.3% <ø> (ø)
... and 18 more

... and 25 files with indirect coverage changes

@eccabay eccabay marked this pull request as ready for review August 17, 2023 13:43
@eccabay eccabay merged commit 41f8e87 into 4266_msts_search Aug 17, 2023
18 of 24 checks passed
@eccabay eccabay deleted the msts_bool_to_problem_type branch August 17, 2023 13:43
eccabay added a commit that referenced this pull request Aug 21, 2023
* Integration into search, with tests

* Swap is_multiseries logic to problem type (#4278)
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