-
Notifications
You must be signed in to change notification settings - Fork 231
⚙️ Add spooky map configs #5020
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
base: master
Are you sure you want to change the base?
Conversation
|
I am not sure that this is the best spot for these.
I wouldn't be worried about these things if the mapper put it in the map themselves, as that is their choice. The old system of taking apart a map and reuploading it at least makes it clear that the new lighting is a derivative rather than the original. Commander hats don't have these considerations because they are active for a whole month, and are pretty easy to miss. On a technical side, shouldn't the config structure just be unchanged? The files themselves can look up the date and decide what to return. |
Good point, it shouldn't run for singleplayer.
There was originally meant to be a tournament but apparently that isn't happening. So it's just for the local-timezone halloween. Perhaps it could span over the closest weekend (I made it single-day mostly because I just dislike this sort of depressing mood specifically).
There was the idea to also have multiple moods (default, Halloween/spooky, Christmas/winter perhaps with snow, night-time, rainy, possibly others) and I wouldn't want to cram all that logic into config files, plus there's the extra logic such as "don't apply to campaign". Or maybe a mission wants to specifically request a specific mood. In that vein I think there should at least be some sort of gameside framework that does most of processing (similar to how there's e.g. |
| local MAPSIDE_FILE = "mapconfig/extraMapSettings.lua" | ||
| local OVERRIDE_CONFIG | ||
|
|
||
| local today = os.date("*t") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this synced? I suspect it is not. Ideally a map-wide event like this would be synced when hosted on the server.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's unsynced. I added synced date to ZKLS recently which sets synced date_day, date_month, and date_year modoptions so it would be doable there but the PR still needs to handle singleplayer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, so local should be a fallback.


See #5015