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.

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

AVERAGE doesn't allow to define numbers as strings #5872

Closed
AMBudnik opened this issue Feb 18, 2019 · 5 comments
Closed

AVERAGE doesn't allow to define numbers as strings #5872

AMBudnik opened this issue Feb 18, 2019 · 5 comments

Comments

@AMBudnik
Copy link
Contributor

Description

AVERAGE doesn't allow to define numbers as strings

Steps to reproduce

  1. Just check the results for Average, Sum, and Min
  2. Only the Average value has an error.

Demo

http://jsfiddle.net/iboylmz/mg067fd8/3/

Your environment

  • Handsontable version: 6.2.2
  • Browser Name and version: Chrome 71
  • Operating System: Windows 10

Moved from GL #139

@AMBudnik AMBudnik transferred this issue from handsontable/handsontable-pro Mar 12, 2019
@AMBudnik AMBudnik added the Part of scope This issue is a part of a larger task label Feb 12, 2021
@AMBudnik
Copy link
Contributor Author

This issue is closed temporarily. It will be reopened for development as it became a part of New formula plugin task reported at #6466

I will make sure to inform everyone interested in this topic after the official fix.

Please feel welcome to makes any comments on this issue. And if you experience similar behavior feel free to contact me at support@handsontable.com

@AMBudnik
Copy link
Contributor Author

This example with pre-v9 version does not change https://jsfiddle.net/handsoncode/tm3rbj1p/

@wojciechczerniak
Copy link
Contributor

This example with pre-v9 version does not change https://jsfiddle.net/handsoncode/tm3rbj1p/

The results are correct? You're calculating empty cells, you can't divide 0 by 0. Did you meant to use C1:C5 ranges?

@AMBudnik
Copy link
Contributor Author

My bad! @wojciechczerniak you're right. There was a typo where the AVERAGE refers to the A column, while it should refer to the C column.

Updated example to pre-v9 https://jsfiddle.net/handsoncode/417toes9/ works well

image

@AMBudnik
Copy link
Contributor Author

AMBudnik commented Jun 2, 2021

Everything also works well in the official v9 (released yesterday) even when we update the values https://jsfiddle.net/3exgnq0y/1/

For everyone interested, here is a list of changes for the following version https://handsontable.com/docs/9.0.0/tutorial-release-notes.html and here https://handsontable.com/docs/9.0.0/tutorial-migration-guide.html is a migration guide (from v8 to v9)

@AMBudnik AMBudnik added Status: Released and removed Part of scope This issue is a part of a larger task labels Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants