Skip to content

Commit

Permalink
[MIRROR] Adds additional type logging to nested feedback for all prin…
Browse files Browse the repository at this point in the history
…ted parts. (#2693)

* Adds additional type logging to nested feedback for all printed parts. (#83466)

## About The Pull Request

What it says on the tin, this adds logging to nested_logging for
autolathes, protolathes, and the mech fabricator.

Also quickly renames a variable name within the mech fabricator because
this thing hasn't been updated in awhile and it was egregious to me.

## Why It's Good For The Game

I wanted to throw together a quick superset chart of what items are
printed most often and how often, and was shocked to learn that it
wasn't logged in this way.

It was recorded somewhat within the silo log, but this is MUCH easier to
use all thing considered, so it's an easy fix.

## Changelog

No front facing changes.

* Adds additional type logging to nested feedback for all printed parts.

---------

Co-authored-by: ArcaneMusic <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
  • Loading branch information
3 people authored and StealsThePRs committed May 27, 2024
1 parent e40514d commit fce9a2d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 7 deletions.
1 change: 1 addition & 0 deletions code/game/machinery/autolathe.dm
Original file line number Diff line number Diff line change
Expand Up @@ -353,6 +353,7 @@
created.pixel_x = created.base_pixel_x + rand(-6, 6)
created.pixel_y = created.base_pixel_y + rand(-6, 6)
created.forceMove(target)
SSblackbox.record_feedback("nested tally", "lathe_printed_items", 1, list("[type]", "[created.type]"))

if(is_stack)
items_remaining = 0
Expand Down
1 change: 1 addition & 0 deletions code/modules/research/machinery/_production.dm
Original file line number Diff line number Diff line change
Expand Up @@ -419,6 +419,7 @@

created.pixel_x = created.base_pixel_x + rand(-6, 6)
created.pixel_y = created.base_pixel_y + rand(-6, 6)
SSblackbox.record_feedback("nested tally", "lathe_printed_items", 1, list("[type]", "[created.type]"))

if(is_stack)
items_remaining = 0
Expand Down
15 changes: 8 additions & 7 deletions code/modules/vehicles/mecha/mech_fabricator.dm
Original file line number Diff line number Diff line change
Expand Up @@ -260,22 +260,23 @@
*
* Returns FALSE is the machine cannot dispense the part on the appropriate turf.
* Return TRUE if the part was successfully dispensed.
* * D - Design datum to attempt to dispense.
* * dispensed_design - Design datum to attempt to dispense.
*/
/obj/machinery/mecha_part_fabricator/proc/dispense_built_part(datum/design/D)
var/obj/item/I = new D.build_path(src)
/obj/machinery/mecha_part_fabricator/proc/dispense_built_part(datum/design/dispensed_design)
var/obj/item/built_part = new dispensed_design.build_path(src)
SSblackbox.record_feedback("nested tally", "lathe_printed_items", 1, list("[type]", "[built_part.type]"))

being_built = null

var/turf/exit = get_step(src,(dir))
if(exit.density)
say("Error! The part outlet is obstructed.")
desc = "It's trying to dispense the fabricated [D.name], but the part outlet is obstructed."
stored_part = I
desc = "It's trying to dispense the fabricated [dispensed_design.name], but the part outlet is obstructed."
stored_part = built_part
return FALSE

say("The fabrication of [I] is now complete.")
I.forceMove(exit)
say("The fabrication of [built_part] is now complete.")
built_part.forceMove(exit)

top_job_id += 1

Expand Down

0 comments on commit fce9a2d

Please sign in to comment.