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

cunumeric/module: implement numpy.meshgrid #587

Open
wants to merge 3 commits into
base: branch-24.03
Choose a base branch
from

Conversation

rohany
Copy link
Member

@rohany rohany commented Sep 17, 2022

This commit implements the function numpy.meshgrid.

This commit implements the function `numpy.meshgrid`.
@rohany rohany requested a review from magnatelee September 17, 2022 17:34
@rohany rohany added the category:new-feature PR introduces a new feature and will be classified as such in release notes label Sep 17, 2022

if not sparse:
# Return the full N-D matrix (not only the 1-D vector).
output = np.broadcast_arrays(*output, subok=True)
Copy link
Member Author

Choose a reason for hiding this comment

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

@magnatelee is there a reason why we haven't implemented broadcast, broadcast_to and broadcast_arrays? If not, I can add them relatively easily as part of this PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is ongoing work #458

Copy link
Member Author

Choose a reason for hiding this comment

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

sweet, i'll just wait for that.


if indexing == "xy" and ndim > 1:
# Switch first and second axis.
output[0].shape = (1, -1) + s0[2:]
Copy link
Member Author

Choose a reason for hiding this comment

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

This implementation from numpy muddles around with the shape parameter of the array. Is there something equivalent that I can do that works on both eager and deferred arrays? I'm not sure the right API call to make.

@manopapad manopapad changed the base branch from branch-22.10 to branch-22.12 September 30, 2022 21:57
@magnatelee
Copy link
Contributor

@rohany can you sign off this pull request? you can force push changes if you need to.

@rohany
Copy link
Member Author

rohany commented Oct 5, 2022

Yes, I will when the PR is ready (it's quite far from that right now).

@marcinz marcinz changed the base branch from branch-22.12 to branch-23.03 January 26, 2023 01:00
@marcinz marcinz changed the base branch from branch-23.03 to branch-23.05 March 6, 2023 20:47
@marcinz marcinz changed the base branch from branch-23.05 to branch-23.07 May 18, 2023 20:30
@marcinz marcinz changed the base branch from branch-23.07 to branch-23.09 July 18, 2023 15:44
@marcinz marcinz changed the base branch from branch-23.09 to branch-23.11 September 26, 2023 00:38
@marcinz marcinz changed the base branch from branch-23.11 to branch-24.01 November 9, 2023 17:14
@marcinz marcinz changed the base branch from branch-24.01 to branch-24.03 February 22, 2024 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-feature PR introduces a new feature and will be classified as such in release notes in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants