Skip to content

Conversation

@paddyroddy
Copy link
Member

Description

We will be using xp etc. throughout the Array API conversion. Currently, changes in this PR have been put in PRs such as #597, but it would make sense that we make these changes separately so downstream PRs only reflect actual Array API changes.

Closes: #599

Checks

  • Is your code passing linting?
  • Is your code passing tests?
  • Have you added additional tests (if required)?
  • Have you modified/extended the documentation (if required)?
  • Have you added a one-liner changelog entry above (if required)?

@paddyroddy paddyroddy added the array-api Work is related to the Array API label Apr 8, 2025
@paddyroddy paddyroddy self-assigned this Apr 8, 2025
Copy link
Member

@Saransh-cpp Saransh-cpp left a comment

Choose a reason for hiding this comment

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

Thanks, @paddyroddy! This looks good!

@paddyroddy
Copy link
Member Author

I'm going to merge this in as it will make Array API related PRs simpler

@paddyroddy paddyroddy merged commit 36bfc44 into main Apr 11, 2025
17 checks passed
@paddyroddy paddyroddy deleted the paddy/issue-599 branch April 11, 2025 07:17
Saransh-cpp pushed a commit that referenced this pull request Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

array-api Work is related to the Array API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace all vars currently called xp with xq

3 participants