Skip to content

Conversation

@matthiasdiener
Copy link
Collaborator

@matthiasdiener matthiasdiener commented Jun 21, 2023

(e.g. np.ndarray)

cc inducer/arraycontext#235

@matthiasdiener matthiasdiener self-assigned this Jun 21, 2023
@matthiasdiener matthiasdiener requested a review from inducer June 21, 2023 19:24
@inducer
Copy link
Owner

inducer commented Jun 21, 2023

I'm not sure why that'd be necessary. For arrays that don't do tagging, wouldn't actx.tag be a no-op?

@matthiasdiener
Copy link
Collaborator Author

matthiasdiener commented Jun 22, 2023

I'm not sure why that'd be necessary. For arrays that don't do tagging, wouldn't actx.tag be a no-op?

actx.tag() is a no-op in inducer/arraycontext#235, but ary.tags won't work (since ary is an ndarray in that case).

@inducer
Copy link
Owner

inducer commented Jun 22, 2023

actx.tag is a no-op in inducer/arraycontext#235, but ary.tags won't work (since ary is an ndarray in that case).

That makes sense. But in that case, only the access ary.tags should be in the try block.

@matthiasdiener
Copy link
Collaborator Author

That makes sense. But in that case, only the access ary.tags should be in the try block.

What do you think of 50f3134?

@inducer inducer merged commit 94f442c into main Jun 22, 2023
@inducer inducer deleted the reshape_and_preserve_tags-untagged branch June 22, 2023 16:10
@inducer
Copy link
Owner

inducer commented Jun 22, 2023

LGTM, thanks!

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.

3 participants