FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

replace usage of `issubclass` with `safe_issubclass` in `BaseDocWitho… by yxtay · Pull Request #1904 · docarray/docarray · GitHub

replace usage of issubclass with safe_issubclass in `BaseDocWitho… - #1904

Merged
JoanFM merged 3 commits into
docarray:mainfrom
yxtay:patch-1
Aug 17, 2024
Merged

replace usage of issubclass with safe_issubclass in `BaseDocWitho…#1904
JoanFM merged 3 commits into
docarray:mainfrom
yxtay:patch-1

Conversation

yxtay commented Aug 14, 2024
edited
Loading

Copy link
Copy Markdown
Contributor
  • replaces usage of issubclass with safe_issubclass in BaseDocWithoutID and MilvusDocumentIndex
  • remaining usage of issubclass are in safe_issubclass function and tests only

Fixes #1903

…utId`

Signed-off-by: YuXuan Tay <wyextay@gmail.com>

JoanFM commented Aug 14, 2024

Copy link
Copy Markdown
Member

Hey @yxtay, I would need you to sign off the commits for the PR to be merged.

yxtay commented Aug 15, 2024

Copy link
Copy Markdown
Contributor Author

Hi @JoanFM, I have amended and signed the commits. Please let me know if any other action is required.

JoanFM commented Aug 15, 2024

Copy link
Copy Markdown
Member

I am currently away from device, but I will check the test errors once I am back

yxtay commented Aug 16, 2024
edited
Loading

Copy link
Copy Markdown
Contributor Author

It looks like many of the failures are due to usage of docker-compose, which has been removed since 2024-04-01.

actions/runner-images#9557

I have created a PR #1905

yxtay and others added 2 commits August 16, 2024 11:02

JoanFM commented Aug 17, 2024

Copy link
Copy Markdown
Member

Thanks for the contribution!

JoanFM merged commit e6802a2 into docarray:main Aug 17, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

Python 3.10 TypeError: issubclass() arg 1 must be a class

2 participants


Back | FazBrowse Home | New Git URL