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

Fix Linting from Pillow version update #719

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

XiaohanZhangCMU
Copy link
Member

Description of changes:

Pillow version is bumped to 10.4.0 from 10.3.0, which results in the following Linting error

tests/test_encodings.py:206:31 - error: Argument of type "int" cannot be assigned to parameter "obj" of type "SupportsArrayInterface" in function "fromarray"

Reason is that fromarray expects np.array while np.random.randint returns Union[int, array]. Wrap with np.array fixes it.

Copy link
Collaborator

@snarayan21 snarayan21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Copy link
Contributor

@b-chu b-chu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@XiaohanZhangCMU XiaohanZhangCMU merged commit d9198ad into main Jul 9, 2024
7 checks passed
@XiaohanZhangCMU XiaohanZhangCMU deleted the xiaohan/fix_pillow_lint branch July 9, 2024 19:19
orionw pushed a commit to orionw/streaming that referenced this pull request Jul 10, 2024
XiaohanZhangCMU added a commit that referenced this pull request Jul 11, 2024
* init commit for hf

* fix reqs

* fix test

* change error name; throw error; fix reqs

* Update docs/source/how_to_guides/configure_cloud_storage_credentials.md

Co-authored-by: Saaketh Narayan <saaketh@mosaicml.com>

* fix test credential failure

* cleanup

* Remove duplicate `dbfs:` prefix from error message (#712)

* fix typo in tests

* docs

* Try to figure out what is wrong with lint; test

* fix os join

* try to fix precommit

* a (#713)

* Upgrade ci_testing, remove codeql (#714)

* linting_codeql

* yo

* yo

* Wrap with nparray (#719)

* Bump pydantic from 2.7.4 to 2.8.2 (#718)

Bumps [pydantic](https://github.com/pydantic/pydantic) from 2.7.4 to 2.8.2.
- [Release notes](https://github.com/pydantic/pydantic/releases)
- [Changelog](https://github.com/pydantic/pydantic/blob/main/HISTORY.md)
- [Commits](pydantic/pydantic@v2.7.4...v2.8.2)

---
updated-dependencies:
- dependency-name: pydantic
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xiaohan Zhang <xiaohanzhang.cmu@gmail.com>

* Bump databricks-sdk from 0.28.0 to 0.29.0 (#715)

Bumps [databricks-sdk](https://github.com/databricks/databricks-sdk-py) from 0.28.0 to 0.29.0.
- [Release notes](https://github.com/databricks/databricks-sdk-py/releases)
- [Changelog](https://github.com/databricks/databricks-sdk-py/blob/main/CHANGELOG.md)
- [Commits](databricks/databricks-sdk-py@v0.28.0...v0.29.0)

---
updated-dependencies:
- dependency-name: databricks-sdk
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Xiaohan Zhang <xiaohanzhang.cmu@gmail.com>

* update docstring

* add isort

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Saaketh Narayan <saaketh@mosaicml.com>
Co-authored-by: Vansh Singh <vansh.singh@databricks.com>
Co-authored-by: bigning <bigning1118@gmail.com>
Co-authored-by: Saaketh Narayan <saaketh.narayan@databricks.com>
Co-authored-by: Xiaohan Zhang <xiaohanzhang.cmu@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

None yet

3 participants