unitystation / unitystation

The original unitystation
https://unitystation.org
GNU Affero General Public License v3.0
704 stars 648 forks source link

Adding Boxstation #3050

Closed ThatDan123 closed 4 years ago

ThatDan123 commented 4 years ago

Purpose

Adds BoxStation to the map pool.

Trying to keep it as authentic as possible, but some changes will need to be made.

Notes:

Still Under Active Development

This PR is to list how far along the mapping is and to list anything that needs to be added for the map to be complete.

TODO: After Merge

Please make sure you have followed the self checks below before submitting a PR:

Picture as of 15/02/2020:

image

ThatDan123 commented 4 years ago

Missing Floor Tiles:

image image

IGNORE BELOW, needs system to support it first.

Also all the decorations on tiles needs to be fixed:

image

image

ThatDan123 commented 4 years ago

Missing Door Prefabs are:

See-through Security

See-through Command

See-through Engineering

See-through Cargo

See-through Atmospherics

See-through Medical

See-through Science

Non-see through Public

AI Doors

Vault Door

craftxbox commented 4 years ago

what variant box is this?

voidel commented 4 years ago

Impressive work.

Misterixmax commented 4 years ago

Oh my god!!! Really impressive, cool. How is it with performance?

ThatDan123 commented 4 years ago

what variant box is this?

Should be Tg, this is my reference image https://tgstation13.org/wiki//images/d/d1/Boxstation.png

Oh my god!!! Really impressive, cool. How is it with performance?

Feels good to me but it hasnt been stress tested eg with plasma

Bod9001 commented 4 years ago

We don't have a system for tile decals yet

DooblyNoobly commented 4 years ago
Lighting is very buggy atm, hard to work with so i've stopped.
Needs a rework, and the ability for lights to function without switches.
50% complete, all lights are in position, just needed reworking light scripts.

Did you add all of the light switches?

ThatDan123 commented 4 years ago

Ive added all the light switches that are on the base tg boxstation map. But there are lots of places which dont require a light switch, it just feels a bit weird having a light switch for some of those places.

Evan-R-Rogers commented 4 years ago

You could disable the Sprite renderer for the light switches to essentially make them invisible as well as hiding them in walls or outside of the station to prevent them from being interacted with.

voidel commented 4 years ago

Seems a little bit shonky vs a solution that creates a new type of light that has no requirment on a switch.

ThatDan123 commented 4 years ago

I've just mapped some temporary light switches.

ThatDan123 commented 4 years ago

The last stuff that needs to be added/fixed is:

Once those are done Box will be in a playable state.

ThatDan123 commented 4 years ago

There's a bug currently which i cant track down, where you cannot place items on tables on the station matrix, tables work fine on the shuttles but not on the station.

DooblyNoobly commented 4 years ago

We can fix up annoying bugs like that for you. How far off are you do you think from removing the DNM?

ThatDan123 commented 4 years ago

I can put temporary doors in later today, then it should be good to merge.

ThatDan123 commented 4 years ago

Doobly you can merge box in, ive added the temporary doors.

Actually hold for a bit, found a bug.

Good to merge.

ThatDan123 commented 4 years ago

You guys reverting and readding the updatemanager changes has broken my fork.

Dont merge this.

ThatDan123 commented 4 years ago

Might have fixed it

ThatDan123 commented 4 years ago

Has been fixed

CodeKalias commented 4 years ago

This map also has the Syndicate spawn point set to Robotics, which apparently breaks nuke rounds (Related to #3210)

AutumnNova commented 4 years ago

it very much does, yeah

ThatDan123 commented 4 years ago

Fixed

fomalsd commented 4 years ago

is this ready for review/merge?

ThatDan123 commented 4 years ago

Yup

DooblyNoobly commented 4 years ago

What changes were made on outpoststation?

ThatDan123 commented 4 years ago

Think my fork is completly messed up, i havnt commited any changes to outpost

ThatDan123 commented 4 years ago

Ive backed up all the files ive changed, and trying to rebase.

ThatDan123 commented 4 years ago

https://github.com/unitystation/unitystation/pull/3232

100% blame doobly and foma for their reverting wars :)