Skip to content

Conversation

@igirault
Copy link
Contributor

WIP: first_proposal

The PR implements meshgrid, based on the API described in #18. It includes tests, spec and examples.

All versions for integer and real types are generated with fypp, as well as multi-dimensional versions, from 1 up to the maximum rank allowed.

The use of the subroutine error_stop in the case of invalid argument indexing prevented me to declare the subroutine with the pure attribute.

WIP: first_proposal
@igirault igirault changed the title meshgrid Add meshgrid subroutine in stdlib_math Jan 18, 2024
@igirault igirault changed the title Add meshgrid subroutine in stdlib_math Add meshgrid subroutine in stdlib_math Jan 18, 2024
Copy link
Member

@jvdp1 jvdp1 left a comment

Choose a reason for hiding this comment

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

Thank you for this PR.
I am not familiar with this. However, the specs, code and tests seem to be good to me. I have only a few suggestions.

@jvdp1 jvdp1 requested review from a team and ivan-pi February 8, 2024 21:38
igirault and others added 4 commits February 9, 2024 13:19
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Copy link
Member

@jvdp1 jvdp1 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @Ivanou34 for this PR.

Copy link
Member

@perazz perazz left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you @Ivanou34

@jvdp1
Copy link
Member

jvdp1 commented Mar 5, 2024

Thank you @perazz for your review. I will fix the conflict ( in a CMake file) and merge it.
Thank you @Ivanou34 for this PR!

@jvdp1 jvdp1 merged commit 8476d65 into fortran-lang:master Mar 5, 2024
@igirault igirault deleted the meshgrid branch April 16, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants