Skip to content

Fixes the timer for restart votes so that it uses real time instead of game time. Also fixes the calendar date to lore#487

Merged
Ossa88 merged 15 commits intoPentestSS13:masterfrom
Constellado:restart-timer-fix
Feb 27, 2026
Merged

Fixes the timer for restart votes so that it uses real time instead of game time. Also fixes the calendar date to lore#487
Ossa88 merged 15 commits intoPentestSS13:masterfrom
Constellado:restart-timer-fix

Conversation

@Constellado
Copy link
Copy Markdown
Collaborator

About The Pull Request

The restart timers are actually set to real time! hurrah! no more restart timers that is supposed to be 3 days ending up going for a week!

The lore edit should fix the year to fit our lore. It should now say 2565 (or 2566 after new years).
I can make the lore edit modular, but I am putting it like this for now!

Why It's Good For The Game

Restart timers actually counts properly

Lore edit good, I think?

Changelog

🆑
add: The in game calendar date is now correct!
fix: Server restart votes should actually be called at the right time now.
/:cl:

…f server time. Also fixes the in game clock to fit our lore.
@Constellado
Copy link
Copy Markdown
Collaborator Author

the calendar year didn't actually change in game I am very confused.

Comment thread code/__HELPERS/time.dm Outdated
Copy link
Copy Markdown
Collaborator

@Ossa88 Ossa88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will properly modularize this a little later, just getting it working for now.

@github-actions github-actions Bot requested a review from Ossa88 January 24, 2026 14:32
@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Jan 24, 2026

Ok, this time I did like 10x the testing.

I had incorrectly assumed some time variables were in normal seconds... nah... deciseconds... so when I started to convert them into different times I did not notice since all my tests were done in small batches.

WELL... because of this I somehow made the code think it was actually... checks calendar... year 2260 because i accidently multiplied the time code by 10... adding hundreds of years.

This version also fixes a flaw I failed to foresee... People voting no... old version would just let the round continue until the next config file said to restart. New version gives ya 30 min + 10 for the jump... to finish RP.... Else if vote passes you get 10 for the jump process.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Jan 24, 2026

Ok, this time I did like 10x the testing.

I had incorrectly assumed some time variables were in normal seconds... nah... deciseconds... so when I started to convert them into different times I did not notice since all my tests were done in small batches.

WELL... because of this I somehow made the code think it was actually... checks calendar... year 2260 because i accidently multiplied the time code by 10... adding hundreds of years.

This version also fixes a flaw I failed to foresee... People voting no... old version would just let the round continue until the next config file said to restart. New version gives ya 30 min + 10 for the jump... to finish RP.... Else if vote passes you get 10 for the jump process.

Also part of the problem, I did not want the server to also include decimal place seconds, so I added my own converter. Well that was setup wrong. This one so far seems to have finally worked correctly and did not jump around too much.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Jan 29, 2026

So far this works, at the conclusion of the round ill work on proper modularization of the code.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Feb 9, 2026

Things to update:

Allow 2 extensions of 30 min.
Remove non voters from the vote pass calculation. Raw actual votes.
Verify how ties are handled. Ties will side with extend, except in 0 v 0 ties.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Feb 12, 2026

Converted from a T/F one time extension to a counter with a 2 maximum currently.

@Ossa88
Copy link
Copy Markdown
Collaborator

Ossa88 commented Feb 26, 2026

Prepping for Merge - Feb 26

@Ossa88 Ossa88 added this pull request to the merge queue Feb 27, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 27, 2026
@Ossa88 Ossa88 added this pull request to the merge queue Feb 27, 2026
@Ossa88 Ossa88 removed this pull request from the merge queue due to the queue being cleared Feb 27, 2026
@Ossa88 Ossa88 added this pull request to the merge queue Feb 27, 2026
@Ossa88 Ossa88 removed this pull request from the merge queue due to the queue being cleared Feb 27, 2026
@Ossa88 Ossa88 added this pull request to the merge queue Feb 27, 2026
Merged via the queue into PentestSS13:master with commit b08f8db Feb 27, 2026
15 checks passed
MrCat15352 pushed a commit to MANKIND-Collective/Pentest that referenced this pull request Mar 20, 2026
…f game time. Also fixes the calendar date to lore (PentestSS13#487)

The restart timers are actually set to real time! hurrah! no more
restart timers that is supposed to be 3 days ending up going for a week!

The lore edit should fix the year to fit our lore. It should now say
2565 (or 2566 after new years).
I can make the lore edit modular, but I am putting it like this for now!

Restart timers actually counts properly

Lore edit good, I think?

:cl:
add: The in game calendar date is now correct!
fix: Server restart votes should actually be called at the right time
now.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Co-authored-by: Ossa88 (SYNAPSE) <MegaMan101@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants