ArgumentError - argument out of range about sidekiq-statistic HOT 14 CLOSED

@jhollinger hello, thanks for report! ❤️
Could you call this command and send me output of this?

redis-cli hgetall sidekiq:statistic

from sidekiq-statistic.

(empty list or set)

from sidekiq-statistic.

Oh that's b/c we're using a sidekiq namespace, so it's sidekiq:sidekiq:statistic. We have HIPAA concerns, so I can't send it to you. Could you tell me what to look for?

from sidekiq-statistic.

@jhollinger, I want to see what string you are use in Time.parse in #formate_date helper method. This error happened when you replace month and day in parsed string.

Thats why I want to ask you show me what contain "2015-11-23:<WorkerName>:last_time" key in redis statistic hash.

from sidekiq-statistic.

Got it. Here's a sampling of last_time values. They're obviously in the American date format. Are they expected to be in the more standard one?

11/17/2015 0750 UTC
11/15/2015 0500 UTC
11/16/2015 0500 UTC
11/22/2015 2342 UTC
11/15/2015 2300 UTC
11/21/2015 2256 UTC
11/15/2015 0500 UTC
11/23/2015 2000 UTC

from sidekiq-statistic.

Sorry, I use this method for formate date in UTC. I think you need to use this method too.

from sidekiq-statistic.

Where is it that I would need to use that method?

from sidekiq-statistic.

ping

from sidekiq-statistic.

pong.
Please save data string in UTC format. You have a problem in parsing your data string in custom format.

from sidekiq-statistic.

Right, I mean what "data string?"

from sidekiq-statistic.

ProTip™: it's easy to store dates in a numeric format Time.now.to_i and then use Time.at(number) to make the timestamp when you need to display it. Numeric means you don't need to worry about string formats or timezones.

from sidekiq-statistic.

+1

from sidekiq-statistic.

@mperham thanks! I think I need to rewrite code 😬

from sidekiq-statistic.

Fixed it by @rbUUbr’s PR. Will close the issue

from sidekiq-statistic.