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

Intervals On Bar Not Updating #6

Closed
zmanowar opened this issue May 4, 2023 · 3 comments
Closed

Intervals On Bar Not Updating #6

zmanowar opened this issue May 4, 2023 · 3 comments

Comments

@zmanowar
Copy link
Owner

zmanowar commented May 4, 2023

From #3:

It was successfully logged, but it was my first random to spawn since the conversion so that seems like a likely explanation. Though it seems like the overlay debug tick counter did update, just not the countdown/intervals. I took this picture at 11:00, which is 47 minutes since the previous random. If you convert the ticks to minutes, you get a little less than 46 minutes (I spent a small amount of time logged out/hopping worlds testing things), so that seemed to update just fine.

image

I'll update my comment to say what happens when the next one spawns.

Originally posted by @samszotkowski in #3 (comment)

@zmanowar
Copy link
Owner Author

zmanowar commented May 4, 2023

Creating a new issue here to track this -- the bar most likely reset but the total "Intervals" counter did not. I think I see the issue. Intervals is based on current logged-in time -- may need a secondary counter for last spawn.

Technically the panel calls getIntervalsSinceLastRandom, which calculates the number of intervals as a basis of logged in time.

I think modifying getCurrentSessionNumIntervals to reference the last spawned time in conjunction with the login time might be the fix. Will probably end up writing a test to check a few things.

@samszotkowski
Copy link

samszotkowski commented May 4, 2023

image (screenshot taken at 12:07)
Most things (overlay tick counter, sidebar countdown and green bar) seem to be working properly after my next random spawned, but the interval counter gets updated to show how many intervals since login, and delta event time recorded in the log shows the time since login (it says 1:04:59, which is not possible considering the random spawned at about 1:05:00 since login) instead of the time since last random.

@zmanowar
Copy link
Owner Author

zmanowar commented May 4, 2023

Most likely solved in 39c6d6e. The entire tracking for secondsSinceLastRandomEvent was janky.

Intervals may still be broken, going to investigate a bit further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants