Skip to content

Array type checking fixes#317

Merged
inducer merged 2 commits into
inducer:mainfrom
majosm:array-type-checking-fixes
Jun 16, 2025
Merged

Array type checking fixes#317
inducer merged 2 commits into
inducer:mainfrom
majosm:array-type-checking-fixes

Conversation

@majosm

@majosm majosm commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator

Some changes to help make the Array protocol compatible with pytato's Array. Depends on inducer/pytato#606.

Comment thread test/test_pytato_arraycontext.py
@majosm

majosm commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator Author

@inducer I'm not sure what to do about this pyopencl stub file thing that pyright wants. Should that be ignored for now?

@inducer

inducer commented Jun 5, 2025

Copy link
Copy Markdown
Owner

@inducer I'm not sure what to do about this pyopencl stub file thing that pyright wants. Should that be ignored for now?

Oddly enough, pyopencl is quite close to picking up types: inducer/pyopencl#833.

@majosm majosm force-pushed the array-type-checking-fixes branch 2 times, most recently from 6b0d338 to 5f41e0f Compare June 6, 2025 20:25
Comment thread arraycontext/context.py
@majosm majosm force-pushed the array-type-checking-fixes branch from 5f41e0f to bffcda0 Compare June 16, 2025 18:34
@majosm majosm marked this pull request as ready for review June 16, 2025 18:47
@majosm majosm requested a review from inducer June 16, 2025 18:47
@inducer inducer merged commit 19bb4fe into inducer:main Jun 16, 2025
11 checks passed
@inducer

inducer commented Jun 16, 2025

Copy link
Copy Markdown
Owner

Thx!

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