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

sync with main repo #3

Merged
merged 14 commits into from
May 31, 2022
Merged

sync with main repo #3

merged 14 commits into from
May 31, 2022

Conversation

jaycee-li
Copy link
Owner

No description provided.

samgoodman and others added 14 commits May 18, 2022 09:19
…rvices (#1235)

Co-authored-by: Sam Goodman <goodmansam@google.com>
* initial work on adding google auth mock

* add anonymouscredentials mock to more tests, run linter

* add auth mock to more test classes

* remove auth mock from two test classes
* feat: support autoscaling metrics when deploying models

* feat: support model deploy to endpoint with autoscaling metrics

* fix autoscaling_target_accelerator_duty_cycle check

* fix docstring: specify that autoscaling_params are optional

* bug fix: add autoscaling_target_cpu_utilization to custom_resource_spec

* add tests

* add _TEST_METRIC_NAME_CPU_UTILIZATION and _TEST_METRIC_NAME_GPU_UTILIZATION

* remove not required arguments in tests

* fix tests: wait for LRO to complete even if not sync

* fix lint: run black

Co-authored-by: Sara Robinson <sararob@users.noreply.github.com>
* feat: add update endpoint

* add validate_traffic and validate_traffic_split

* remove validation, add system tests

* Text fixes

* Nox blacken change

Co-authored-by: Sam Goodman <goodmansam@google.com>
…1254)

* replace filepath in test with tmpdir var

* update script path var in training job tests

* run linter
Release-As: 1.13.1

Co-authored-by: Sam Goodman <goodmansam@google.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Adds e2e test for AutoML Forecasting and unit test for `TimeSeriesDataset`. Also adds `create_request_timeout` to `TimeSeriesDataset`, which #1099 seems to have missed.

---

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-aiplatform/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕
… job (#1255)

Adds support for hierarchical forecasting and window filtering.

---

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-aiplatform/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes b/229907889 b/228499154 🦕
* chore: release 1.13.1

Release-As: 1.13.1

* Changed test to use list_models() correctly

* feat: Changed system test to use list_models() correctly

Co-authored-by: Sam Goodman <goodmansam@google.com>
Add holiday regions for Vertex Forecasting.

---

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-aiplatform/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes b/232964228 🦕
@jaycee-li jaycee-li merged commit 11b0b3d into jaycee-li:main May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants