Skip to content

allowing tuple datatype for the 1st param of array_map #907

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 6, 2025
Merged

allowing tuple datatype for the 1st param of array_map #907

merged 2 commits into from
Mar 6, 2025

Conversation

Jasmine-ge
Copy link
Contributor

fix arrayMap with array of tuples with single argument

PR checklist:

  • Did you run ClangFormat ?
  • Did you separate headers to a different section in existing community code base ?
  • Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:
Support tuple as 1st param of array map function. close #906

@Jasmine-ge
allowing tuple datatype for the 1st param of array_map
8124ca4 
fix arrayMap with array of tuples with single argument
@Jasmine-ge Jasmine-ge requested a review from yl-lisen February 20, 2025 06:37
@Jasmine-ge Jasmine-ge self-assigned this Feb 20, 2025
@Jasmine-ge Jasmine-ge marked this pull request as ready for review February 20, 2025 06:37
@yl-lisen yl-lisen merged commit f087cf2 into develop Mar 6, 2025
21 checks passed
@yl-lisen yl-lisen deleted the enhancement/issue-906-1st-param-of-array-map branch March 6, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yl-lisen yl-lisen yl-lisen approved these changes

Assignees

@Jasmine-ge Jasmine-ge

Labels
pr-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1st param of array_map cannot be a tuple
2 participants
@Jasmine-ge @yl-lisen