+
Skip to content

Conversation

Googles-Hands
Copy link
Contributor

@Googles-Hands Googles-Hands commented Oct 4, 2025

About the pull request

This adds a colony blackbox + CIR to New Varadero.

Explain why it's good for the game

It's a neat thing to have that Liaisons, Command and the intelligence team can use as a roleplay object and side objective, they were more than likely simply forgotten to be added rather than purposefully left out.

Testing Photographs and Procedure

Screenshots & Videos Blackbox1

Changelog

🆑
maptweak: A blackbox and colony information recorder have been added to New Varadero
/:cl:

This adds colony blackboxes + CIR's to the two maps missing them.
@Googles-Hands Googles-Hands requested a review from Nanu308 as a code owner October 4, 2025 12:28
@github-project-automation github-project-automation bot moved this to Awaiting Review in Review Backlog Oct 4, 2025
@cmss13-ci cmss13-ci bot added Mapping did you remember to save in tgm format? size/S Denotes a PR that changes 19-49 lines, ignoring generated files. labels Oct 4, 2025
c3bb8dc: maps/map_files/New_Varadero/New_Varadero.dmm

Automatically commited by: tools\mapmerge2\fixup.py
@Zenith00000
Copy link
Contributor

There's already two on Hybrisa, and having that box in the middle of the offices looks weird, doesn't fit.

Also I like the idea of them actually providing some intel perhaps, or some actual gameplay benefit, other then RP, so might be a good idea to actually make them do something, but yeah that would be another PR.

@Googles-Hands
Copy link
Contributor Author

Googles-Hands commented Oct 4, 2025

There's already two on Hybrisa, and having that box in the middle of the offices looks weird, doesn't fit.

Also I like the idea of them actually providing some intel perhaps, or some actual gameplay benefit, other then RP, so might be a good idea to actually make them do something, but yeah that would be another PR.

You are right, I just searched for the big blackboxes and presumed those that miss that also miss the little ones. I'll ammend the PR in a couple of hours.

As for the gameplay suggestion, I also had an idea for something like that, with them being able to be decoded in the ARES core directly for some intel and a lore-paper like the Liaison gets at roundstart depending on the map that would show some randomised logs of colony announcements/activity during the breach. However I'm just an ideasguy in that regard.

@Zenith00000
Copy link
Contributor

There's already two on Hybrisa, and having that box in the middle of the offices looks weird, doesn't fit.
Also I like the idea of them actually providing some intel perhaps, or some actual gameplay benefit, other then RP, so might be a good idea to actually make them do something, but yeah that would be another PR.

You are right, I just searched for the big blackboxes and presumed those that miss that also miss the little ones. I'll ammend the PR in a couple of hours.

As for the gameplay suggestion, I also had an idea for something like that, with them being able to be decoded in the ARES core directly for some intel and a lore-paper like the Liaison gets at roundstart depending on the map that would show some randomised logs of colony announcements/activity during the breach. However I'm just an ideasguy in that regard.

The least that could be done is adding "desc_lore =" to them on each map, with an appropriate lore-thing written out perhaps? Other stuff would be cool too, but yeah I know this wasn't the original intent of the PR, but expanding it could be cool. Just a suggestion.

@cmss13-ci cmss13-ci bot added size/XS Denotes a PR that changes 0-19 lines, ignoring generated files. and removed size/S Denotes a PR that changes 19-49 lines, ignoring generated files. labels Oct 4, 2025
@Googles-Hands Googles-Hands changed the title Adds colony blackboxes to New Varadero and LV759 Adds colony blackboxes to New Varadero Oct 4, 2025
@Googles-Hands
Copy link
Contributor Author

The least that could be done is adding "desc_lore =" to them on each map, with an appropriate lore-thing written out perhaps? Other stuff would be cool too, but yeah I know this wasn't the original intent of the PR, but expanding it could be cool. Just a suggestion.

Do you mean the CIR's?

@Zenith00000
Copy link
Contributor

The least that could be done is adding "desc_lore =" to them on each map, with an appropriate lore-thing written out perhaps? Other stuff would be cool too, but yeah I know this wasn't the original intent of the PR, but expanding it could be cool. Just a suggestion.

Do you mean the CIR's?

Yeah, you can basically add an extended description to them, which could be pretty long, if you wanted to write out some sort of colony lore thing.

@Steelpoint
Copy link
Contributor

If you are intending to expand the PR by adding more descriptors you should draft the PR, if you are finished with this, I'll do a review soon.

@Googles-Hands
Copy link
Contributor Author

If you are intending to expand the PR by adding more descriptors you should draft the PR, if you are finished with this, I'll do a review soon.

I'm not in the discord, so I'm not communicating to the loreheads or whoever they have in charge of that. Maybe I'll make a separate PR, but it won't happen in this one.

@Steelpoint
Copy link
Contributor

Mapping LGTM

@Steelpoint Steelpoint added the Mapping Approved adds 500 new dict keys label Oct 8, 2025
@github-project-automation github-project-automation bot moved this from Awaiting Review to Partially Approved / Mergeable in Review Backlog Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format? size/XS Denotes a PR that changes 0-19 lines, ignoring generated files.
Projects
Status: Partially Approved / Mergeable
Development

Successfully merging this pull request may close these issues.

3 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载