search

ProjectZetta / RemoteFutures

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

Issue#34 hopefully resolved #44

Closed ghost closed 10 years ago

ghost commented 10 years ago

@MartinSenne

patched localization stuff. Please merge, puil and run test aka:

project benchmark test

In case the test passes, please look into the Result folder. there should be a file "test.xls"

if not, please send me a gist with the output.

Thank you

MartinSenne commented 10 years ago

Thx for your efford, Marvin!!!! I'll merge now and test tomorrow. Ya, M$ and localization ....... %-/ If it is not working tomorrow, I'll try a different locale setting.

Cheers, Martin

ghost commented 10 years ago

Keep smilling,

all will be good.

Locale is set as constant to EN in ExcleWriter in util package. Also, I've wrote a small tool for putting all local setting in the test output! Just scroll console up and you'll see.

Having said, I try to avoid localizing the generated excel sheet by setting the global worksheet local to EN and keep everything the way it was before.

Hope that works.

marvin -- send from mobile phone On 1/04/2014 9:26 PM, "Martin Senne" notifications@github.com wrote:

Thx for your efford, Marvin!!!! I'll merge now and test tomorrow. Ya, M$ and localization ....... %-/ If it is not working tomorrow, I'll try a different locale setting.

Cheers, Martin

— Reply to this email directly or view it on GitHubhttps://github.com/DistributedRemoteFutures/DistributedRemoteFutures/pull/44#issuecomment-39247672 .

MartinSenne commented 10 years ago

Works! Good job!

ghost commented 10 years ago

Thanks,

Just one last thing. Please open the generated excel file and have a look if all stats are calculated. The data are randomly generated but min, max, median and stdv should be there if it works correctly.

Cheers

marvin -- send from mobile phone On 2/04/2014 9:51 AM, "Martin Senne" notifications@github.com wrote:

Works! Good job!

— Reply to this email directly or view it on GitHubhttps://github.com/DistributedRemoteFutures/DistributedRemoteFutures/pull/44#issuecomment-39297515 .