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.

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

Vec\sum()? #186

Closed
yivi opened this issue Apr 28, 2021 · 4 comments
Closed

Vec\sum()? #186

yivi opened this issue Apr 28, 2021 · 4 comments
Assignees
@azjezz
Labels
Type: Question A query or seeking clarification on parts of the spec. Probably doesn't need the attention of everyo

Comments

@yivi
Copy link
Contributor

yivi commented Apr 28, 2021

I see that there is no array_sum() equivalent provided.

Is this intentional, or is something that you'd be willing to consider?
@yivi yivi added the Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. label Apr 28, 2021
@yivi
Copy link
Contributor Author

yivi commented Apr 28, 2021

F*ck. Mislabelled "bug".

@azjezz azjezz added Type: Enhancement Most issues will probably ask for additions or changes. Type: Question A query or seeking clarification on parts of the spec. Probably doesn't need the attention of everyo and removed Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. Type: Enhancement Most issues will probably ask for additions or changes. labels Apr 28, 2021
@azjezz
Copy link
Owner

azjezz commented Apr 28, 2021

This function already exist as Math\sum ( https://github.com/azjezz/psl/blob/1.7.x/src/Psl/Math/sum.php#L14 ), or are you looking for a different behavior?

@azjezz
Copy link
Owner

azjezz commented Apr 28, 2021

there's also https://github.com/azjezz/psl/blob/1.7.x/src/Psl/Math/sum_floats.php#L14 for floats :)

@yivi
Copy link
Contributor Author

yivi commented Apr 28, 2021 •
edited
Loading

Thanks! I was looking in the wrong place, following "core" "logic" for this.

@yivi yivi closed this as completed Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azjezz azjezz

Labels
Type: Question A query or seeking clarification on parts of the spec. Probably doesn't need the attention of everyo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yivi @azjezz