Skip to content
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

[Maps] Dev rework #34708

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

Ko4ergaPunk
Copy link
Contributor

@Ko4ergaPunk Ko4ergaPunk commented Jan 28, 2025

About the PR

Rework of Dev map.

Why / Balance

The current version has a big problem. Sections further than five meters from spawn do not make any sense. Nobody runs at a snail's pace to the R&D department for unlock technology, or to Med for clone Urist. Everyone just use spawn menu. I think that's too silly and propose a solution. Just use portals for fast travel. Also, this solution will allow to expand Dev without problems if it's necessary.

Media

Dev-0

dev.portals.demo.mp4

Requirements

Changelog
no cl

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XL Denotes a PR that changes 5000+ lines. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Map Changes: Might require knowledge of mapping. labels Jan 28, 2025
@Killerqu00
Copy link
Contributor

HUGE

@ArtisticRoomba ArtisticRoomba added P3: Standard Priority: Default priority for repository items. T: New Feature Type: New feature or content, or extending existing content S: Needs Review Status: Requires additional reviews before being fully accepted DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. A: Mapping Area: Maps, or tools used for mapping and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Jan 28, 2025
@HerCoyote23
Copy link
Contributor

Looks sweet, but you should probably rename the "Polygon" portal into "Firing Range" or "Weapons" or something similar, as polygons have nothing to do with what's in that room.

@slarticodefast
Copy link
Member

slarticodefast commented Jan 28, 2025

Should have more AI cameras in my opinon, so we can test the AI interacting with everything.
Also this does not need an admin changelog, most admins are not game devs.

@Ko4ergaPunk
Copy link
Contributor Author

Should have more AI cameras in my opinon, so we can test the AI interacting with everything. Also this does not need an admin changelog, most admins are not game devs.

Should I remove ADMIN or all cl? idk

@lzk228
Copy link
Contributor

lzk228 commented Jan 29, 2025

all cl

@slarticodefast
Copy link
Member

slarticodefast commented Jan 29, 2025

Some more requests from me:
grafik
Add some more tables and space the items out a little like on the current dev map, so they don't overlap. Add an area with machine frames in different construction steps and a table with all the parts needed to construct different machines.
grafik
grafik
Space out the items here as well.
grafik
Add the melee weapons that are on the current dev map (the captains sabre, a spear, a stun baton). Add a health scanner for damage inspection. Add handcuffs. Add a debug uplink radio.
grafik
Add some pre-spawned produce.

Maybe add a singulo/tesla setup with a PA.
Add a cargo department with a docking port, telepad, the consoles and move the salvage magnet there so you can test it.
I can't find the telecoms and crew monitoring server on the preview image.

@FairlySadPanda
Copy link
Contributor

This probably needs an AI to test that code with. Would also prefer a testing area for doors, door hacking, etc.

@Ko4ergaPunk Ko4ergaPunk marked this pull request as draft January 30, 2025 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Mapping Area: Maps, or tools used for mapping Changes: Map Changes: Might require knowledge of mapping. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted size/XL Denotes a PR that changes 5000+ lines. T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants