* Add SScamera
* Refactor camera
* Refactor how camera chunks add/remove cams
* Use CAMERA_VIEW_DISTANCE to determine max view dist of cam
* Fix movable cameras and optimise them
* Optimise update()
* Ensure aiEyes get removed properly when deleted
* Final optimisations and refactoring
* Fix cameras being EMPed
* Shits fixed
* AA review
* Fix merge mistake
* Add comment about early return
* Seans review
* the it the
Co-authored-by: S34N <12197162+S34NW@users.noreply.github.com>
* Update code/game/machinery/camera/camera.dm
Co-authored-by: Ryan <80364400+Sirryan2002@users.noreply.github.com>
* Ensure the robit has a camera before updating it
---------
Co-authored-by: S34N <12197162+S34NW@users.noreply.github.com>
Co-authored-by: Charlie <69320440+hal9000PR@users.noreply.github.com>
Co-authored-by: Ryan <80364400+Sirryan2002@users.noreply.github.com>
* Fix ID consoles denying management access to head IDs witih custom titles
* Update proc name
* Make ID consoles highlight current rank regardless of custom title
* Fix ID consoles improperly checking Demoted/Terminated status
* Fix un-terminations not requiring a full ID console
* Fix missing `selected` properties; readability
* Fix console not disabling the Demote option on already demoted cards
* Deduplicate full and departmental ID console UI code
* Move the tab-agnostic authentication UI above tabs
* Angry feedback message consistency
* I forget this shorthand operator exists
* Some verbs queue up now
* Damn you 515
* Screw you Topic
* Update client_procs.dm
* Fixes for unit testing
* Update MC.dm
* verb
* Update callback.dm
* Tweaks
* Linters might not have liked that,
* More tweaks, and fix
* Update verb_manager.dm
* I AM NOT A KING. I AM NOT A GOD. I AM LUNACY.
* THE SCOPE CREEP
* This was done in vi. No langserver. Send help.
* Progress
* Some issues fixed
* The code builds, but the maps dont
* CC builds, I doubt anything else does
* Do this now
* Safety
* The map compiles
* THE MAPPING HELL
* Airlock controllers dont work
* Fixed bombs
* AND THATS FIXED
* Phantom edit
* Restores LL I think
* The mapmerge hooks are actually retarded
* Fixes spawners
* Half working air control
* Picky
* Hyper efficient tank monitor
* Sanity
* Laying the framework
* Improved alert console
* Dont think this is actually done anywhere
* Metering
* ONE HUNDRED AND ~~EIGHTY~~ TWENTY SEVEN
* WE ARE READY FOR BOX TO BUILD
* One map done
* Well that was easy
* Another one
* I think this is done
* I should have removed this
* I would make a russian joke but <current event>
* Delta WIP
* Makes delta work
* Early review
* TM safeguard
* oops
* METAAAAAAAAAAAAAAAAAAAAAAAAA
* Fix#4213
* Trailing
* oh for piss sake
* Shutter fix
* Roundstart ATs can go away
* Review pass 1
* What could go wrong
* BOOM BOOM BOOM
* Not needed
* Fix seed vault
* Oops
* Review changes
* Implement map tests for catching common errors.
- Adds test runner:
- to make it easier to track things across test types
- for example to ensure a fully specified log can be emitted
- Adds map tile test type:
- when writing a test, coders implement CheckTile, which is
handed a single turf
- when the test runner runs these tests, it iterates over
all turfs in the specified z-level, and runs each test's
CheckTile on each turf in turn.
- Adds two sample map tile tests:
- check to see if a pipe exists on the same tile as a scrubber
or vent
- check to see if a tile contains two cables, each with a center
node
* Review #1:
- Replace nested loops over map tiles with `block`
- Remove check for valid turf in individual tests, I think it's safe to
assume `block` will always return legit turfs
- Added proper duration tracking for old tests
- Gave log file an appropriate extension
- Actually use `Fail` for tests
* whoops
* add more tests suggested by @Vi3trice
* Add some more tests courtesy @Bm0n and @Vi3trice
* windows are okay in space as long as it's nearspace
* Add failure threshold to prevent excessive logging.
Once this threshold is reached, a test will stop being processed for
every tile.
Note that this applies to `log_world` and `text2file` equally when
logging large amounts of failures.
* Document each test.
* Remove unnecessary reboot
* Let all map tests run to completion in CI matrix.
* I know what alphabetical means
The way """we""" currently do it, each time you want to walk the queue
you're forced to make a copy in memory of the whole thing
There's no real reason to want this, so it seems best to just avoid it
entirely. It creates a TON of usage for no reason, and also risks a lot
of overtime since you can't really batch a list copy like that.
So instead let's just iterate over the length of the queue, constant
rather then O(N) time.
Similarly, rather then using an associated list in the form queue[ref] = gc time,
we could store queue entries in what amounts to a tuple.
This means no associated list stuff, so the operation of queuing becomes
cheaper, and pulling gc time similarly goes from O(log n) to constant time
I stole this work from myself and mso, tg pr 55595
I'm pring it here because I keep seeing affected complain about the
garbage subsystem and he refuses to do it himself. No I don't have an
ego problem I swear
* Refactor announcements.
* Actually make this a unit test
* Dumb
* Remove unnecessary 'as text'
* Reviews:
- Make CSS class a recognizable word
- Flip the defaults for logging
Also:
- Style fixes (default null in fields)
- Fix emergency/crew-transfer shuttle announcement titles
* move along move along
* Update bot.dm
* Diagonals are now more expensive
* Update path.dm
* Update parrot.dm
* Update path.dm
* Tweaks
* Fix cleanbot, add path safety
* Tweaked, added a safety, removed the previous one
* Update medbot.dm
* path.len isn't very safe as a whole, floorbots also had order of operations wrong
* Update medbot.dm
* clings not even once
* Back to the drawing board
* Update path.dm
* Make mules actually clear the drawn path.
* Make bots use step_towards unconditionally instead of flipping between step_to and Move
* Making extra sure the path is cleared. Somehow path was left over.
* Check for length as get_path_to is always true
* This and that
Removes a line of code that does... nothing! Causes a runtime on world starting so you need to allow it to execute the rest, and I hate being forced to click 1 extra time. Grug like fast debugger, not slow debugger.
