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

Enable isinstance to work with PyJClass. #440

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

bsteffensmeier
Copy link
Member

This fixes #438

I'm not sure if this will go in 4.1.1 or 4.2, since we haven't created a dev_4.2 branch yet I am currently targeting dev_4.1 but I may wait to merge this until we decide on a release plan.

@ctrueden
Copy link
Contributor

This is great. To work around this limitation with the currently released version of Jep, I was doing isinstance(obj, jtype.__pytype__); it'll be great to be able to use isinstance(obj, jtype) directly instead.

@bsteffensmeier bsteffensmeier merged commit 9083c70 into ninia:dev_4.1 Nov 30, 2022
@bsteffensmeier
Copy link
Member Author

This will be included in the 4.1.1 release which I am planning to put out around the end of December.

@ndjensen
Copy link
Member

I forgot to mention, I think you should add an entry in the 4.1 release notes that points out isinstance now works as one might expect.

@bsteffensmeier bsteffensmeier deleted the pyjclass-isinstance branch May 16, 2023 02:44
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