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

"Invalid array length" bug during response transpose #11

Open
knownasilya opened this issue Dec 21, 2018 · 2 comments
Open

"Invalid array length" bug during response transpose #11

knownasilya opened this issue Dec 21, 2018 · 2 comments

Comments

@knownasilya
Copy link
Contributor

This code returns Infinity, but it should never https://github.com/AppGeo/maplarge/blob/master/lib/index.js#L364

screen shot 2018-12-21 at 3 16 02 pm
screen shot 2018-12-21 at 3 16 10 pm
@calvinmetcalf
Copy link
Member

yeah so looks like it doesn't take into account that the array might be empty

@knownasilya
Copy link
Contributor Author

Maybe default value for that reduce should be 0?

calvinmetcalf added a commit that referenced this issue Dec 21, 2018
@calvinmetcalf
check for infinity. fixes #11
83c5554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@knownasilya @calvinmetcalf