Skip to content

Conversation

@josdejong
Copy link
Owner

See #1771

Copy link
Collaborator

@harrysarson harrysarson left a comment

Choose a reason for hiding this comment

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

Looks good!

/**
* Calculate the size of a matrix or scalar.
*
* The function always returns an Array, also for Matrix input.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* The function always returns an Array, also for Matrix input.
* The function always returns an Array, even if the input is a Matrix.

@josdejong
Copy link
Owner Author

Looks good!

Thanks for checking it out Harry. We're not sure yet though if we do want this change, see #1771 (if you have any thoughts I would love to hear them in #1771).

@josdejong josdejong closed this May 27, 2020
@josdejong josdejong deleted the feature/size_as_array branch May 31, 2024 14:02
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