ProjectZetta / RemoteFutures

Remote execution of futures in a distributed system.
http://remotefutures.org
Other
4 stars 1 forks source link

FIX: Unrecognized function when parsing formula AVERAGE(A1:A101) #34

Closed ghost closed 10 years ago

MartinSenne commented 10 years ago

Works like a charm now! Thanks for fixing this!

ghost commented 10 years ago

Good,

Just to let you know, I'm away for the rest of the day.

Cheers marvin On 28/03/2014 4:41 PM, "Martin Senne" notifications@github.com wrote:

Works like a charm now! Thanks for fixing this!

— Reply to this email directly or view it on GitHubhttps://github.com/DistributedRemoteFutures/DistributedRemoteFutures/issues/34#issuecomment-38933285 .

ghost commented 10 years ago

Need to investigate the actual cause of the issue;

ghost commented 10 years ago

Still working on isolating the cause. Apparently, its not trivial to reproduce that error.

ghost commented 10 years ago

Travis builds and tests correctly. So there is no obvious issue with the generated Excel sheets. However, just worth asking the "dumb" question:

Do you use German as system or Office language?

I'm not joking. I try to explain this as careful as possible. Apparently, there is an issue with local settings[1] which means:

AVERAGE(A1:A101)

should be

MITTELWERT(A1:A101)

in order to work on systems with non-English language settings.

If that's the case, I would close this issue with a corresponding notice.

[1] Guess who made that crap.... http://office.microsoft.com/de-ch/excel-help/berechnen-des-durchschnitts-von-zahlen-HP003056135.aspx