-
Notifications
You must be signed in to change notification settings - Fork 8
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
Bad time passed calculation when <24 hours #99
Comments
Hmmm. There have been some bugs with the date picker in the past so this may just be a reoccurrence of that, which I know how to fix. I'll look into it! |
Sorry about that, I fixed this issue in the first commit of #96, but I can also separate it out into a different PR. |
Oh that's ok! I'll look over #96 when I get a chance to! |
I've noticed the same issue, though for me it persists for addiction created at any time. (limited testing, tried ~20 hours, ~2 days, ~1 week) they all seems to add exactly 1 day. I've also pretty much confirmed that it's not an issue with the date picker, as I exported data, and the epoch timestamps are all exactly as they should be. I poked around in your codebase briefly, but I'm not particularly familiar with Kotlin. I would assume it's mostly with displaying, or perhaps an odd rounding error, but I have little proof of this |
@KiARC |
I'm really sorry for not putting up 10.1.0 yet. The machine that had the signing keys died a while back and I still have not been able to recover them. The past several months have been swamped with schoolwork, but I finally might have a chance to figure this out. Again, I'm sorry for this getting delayed so much.
|
The problem seems to be that it starts counting at midnight, rather than when hours is greater than 24. Same for when hours would tick up past 24, the day doesn't actually increment until midnight. I'm guessing it was using a built in function, and it counted days by date rather than by hour. |
As far as I'm aware, yes, that's what's happening |
Unfortunately I don't have the keys. I really wish I could work on Sobriety right now but I genuinely just don't have the energy to these days. Android development is exhausting. I very well might get back into it eventually, but that's not realistically going to be soon. I'm really sorry. |
I have been seriously considering a new app, either a PWA or something based on Tauri, because I'm much more comfortable with React than with Jetpack Compose and the rest of the Android ecosystem. That's still kinda up in the air, so don't hold your breath, but if can get that off the ground that would be a lot easier for me than continuing with native Android development. |
Installed fresh version from F-DROID today.
It seems that when I set a new addiction and mark a start time yesterday afternoon, it adds a day to the calculation.
It was 09:00 (9am) today (6th) and I set an addiction start to 23:20 (11:23pm) last night (5th). It showed me 1 day and 9 hours, as it would elapse another day since that.
The text was updated successfully, but these errors were encountered: