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

Wasteplanet Pandora Remap (REQUIRES #1945) #2022

Closed
wants to merge 24 commits into from

Conversation

Zevotech
Copy link
Contributor

@Zevotech Zevotech commented Jun 6, 2023

About The Pull Request

Remaps wasteplanet_pandora to not be a tiny box with a one-way entry into a locked room with a pandora. Requires #1945 to be merged or at least test merged.
2023-07-07 18 22 40
image

Why It's Good For The Game

shit ruin must remap

Changelog

🆑
Add: Remapped wasteplanet_pandora
/:cl:

@github-actions github-actions bot added Code change Watch something violently break. DME Edit labels Jun 6, 2023
@Zevotech Zevotech requested a review from a team as a code owner June 6, 2023 20:57
@github-actions github-actions bot added the Map Change Tile placing is hard. Thank you for your service. label Jun 6, 2023
@Zevotech
Copy link
Contributor Author

Zevotech commented Jun 6, 2023

is there genuinely a fucking check dedicated to the spelling of centcom? im dying.

@Valorium-RB
Copy link

The note(s) used are a bit too brazen, methinks. If it were wrapped in about three pounds of corpo-speak and less overtly colonialist then it'd be about fine.

@Zevotech
Copy link
Contributor Author

Zevotech commented Jun 7, 2023

The note(s) used are a bit too brazen, methinks. If it were wrapped in about three pounds of corpo-speak and less overtly colonialist then it'd be about fine.

its meant to be old "practically a government" nanotrasen so thats what i went for. examples of how to improve it are appreciated though.

@Latentish
Copy link
Member

The note(s) used are a bit too brazen, methinks. If it were wrapped in about three pounds of corpo-speak and less overtly colonialist then it'd be about fine.

its meant to be old "practically a government" nanotrasen so thats what i went for. examples of how to improve it are appreciated though.

Remove the colonial racism, make the note less edgy.
Or remove it entirely. It doesn't totally need to be there.

@Zevotech
Copy link
Contributor Author

Zevotech commented Jun 7, 2023

The note(s) used are a bit too brazen, methinks. If it were wrapped in about three pounds of corpo-speak and less overtly colonialist then it'd be about fine.

its meant to be old "practically a government" nanotrasen so thats what i went for. examples of how to improve it are appreciated though.

Remove the colonial racism, make the note less edgy. Or remove it entirely. It doesn't totally need to be there.

yk what good point. entirely environmental storytelling it is.

@Zevotech
Copy link
Contributor Author

Zevotech commented Jun 7, 2023

up to date and tm ready

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Jun 9, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2023

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Erikafox
Copy link
Contributor

resolve your merge conflicts

@PrefabQuasar
Copy link
Contributor

looks like a large improvement over the original, once merge conflicts are fixed, id like to see it TM'd

Signed-off-by: zevo <95449138+Zevotech@users.noreply.github.com>
@github-actions github-actions bot removed Merge Conflict Use Git Hooks, you're welcome. DME Edit labels Jun 14, 2023
@Zevotech
Copy link
Contributor Author

that should handle the conflicts. not sure why github is inserting the file comparison marks directly into the code but at least its easily fixed

@Latentish
Copy link
Member

Please update your images.
Also I agree with prefab, tm'able.

@Latentish Latentish added the Test Merge Candidate You do it for free. label Jun 14, 2023
Copy link
Member

@Latentish Latentish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this. I tested it in game, it's pretty good. I like the arena.

/turf/open/water/waste/lit
light_range = 2
light_power = 0.2
light_color = LIGHT_COLOR_LAVA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
light_color = LIGHT_COLOR_LAVA
light_color = LIGHT_COLOR_FLARE

/turf/open/floor/plating/dirt/old/waste/lit
light_range = 2
light_power = 0.2
light_color = LIGHT_COLOR_LAVA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
light_color = LIGHT_COLOR_LAVA
light_color = LIGHT_COLOR_FLARE

/turf/open/floor/plating/grass/wasteplanet/lit
light_range = 2
light_power = 0.2
light_color = LIGHT_COLOR_LAVA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
light_color = LIGHT_COLOR_LAVA
light_color = LIGHT_COLOR_FLARE

initial_gas_mix = OPENTURF_DEFAULT_ATMOS
planetary_atmos = TRUE

/turf/open/water/waste/lit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

define the lits where the rest of them are defined in file. That is to say at the bottom.

@@ -48,4 +53,45 @@
/turf/open/water/tar/waste/lit
light_range = 2
light_power = 0.2
light_color = LIGHT_COLOR_LAVA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
light_color = LIGHT_COLOR_LAVA
light_color = LIGHT_COLOR_FLARE

Copy link
Contributor

@Erikafox Erikafox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If something has planetary atmos it needs to match the atmosphere of the planet it's on, otherwise you're just making atmos lag fnr.

light_color = LIGHT_COLOR_LAVA

/turf/open/floor/plating/dirt/old/waste
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also:

Suggested change
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
initial_gas_mix = WASTEPLANET_DEFAULT_ATMOS

light_color = LIGHT_COLOR_LAVA

/turf/open/floor/wood/waste
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
initial_gas_mix = WASTEPLANET_DEFAULT_ATMOS

planetary_atmos = TRUE

/turf/open/indestructible/hierophant/waste
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
initial_gas_mix = WASTEPLANET_DEFAULT_ATMOS

planetary_atmos = TRUE

/turf/open/indestructible/hierophant/two/waste
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
initial_gas_mix = OPENTURF_DEFAULT_ATMOS
initial_gas_mix = WASTEPLANET_DEFAULT_ATMOS

@Zevotech
Copy link
Contributor Author

Zevotech commented Jul 7, 2023

images updated and reviews addressed

//open turfs then open lits.

/turf/open/floor/plating/wasteplanet/lit
light_range = 2
light_power = 0.2
light_color = LIGHT_COLOR_FLARE

/turf/open/floor/plating/rust/wasteplanet/lit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you need to scrub this out. the already defined path is /plating/wasteplanet/rust/lit

@@ -368,6 +374,7 @@

/datum/biome/cave/waste/metal //deeper in, there's no normal stuff here
open_turf_types = list(
/turf/open/floor/plating/rust/wasteplanet = 10,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -334,6 +339,7 @@
/datum/biome/cave/waste/tar_bed //tar colorings here
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet = 70,
/turf/open/floor/plating/rust/wasteplanet = 20,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -274,6 +278,7 @@
/datum/biome/cave/waste
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet = 80,
/turf/open/floor/plating/rust/wasteplanet = 15,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -265,6 +268,7 @@
/datum/biome/waste/metal/rust
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet/lit = 1,
/turf/open/floor/plating/rust/wasteplanet/lit = 10,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -220,6 +222,7 @@
/datum/biome/waste/metal
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet/lit = 5,
/turf/open/floor/plating/rust/wasteplanet/lit = 45,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -208,6 +209,7 @@
/datum/biome/waste/tar_bed //tar colorings
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet/lit = 70,
/turf/open/floor/plating/rust/wasteplanet/lit = 10,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@@ -154,6 +154,7 @@
/datum/biome/waste/crater
open_turf_types = list(
/turf/open/floor/plating/asteroid/wasteplanet/lit = 90,
/turf/open/floor/plating/rust/wasteplanet/lit = 10,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

@Erikafox Erikafox removed the Test Merge Candidate You do it for free. label Aug 5, 2023
@Erikafox
Copy link
Contributor

Erikafox commented Aug 5, 2023

ping us when you feel like addressing the review I left.

@Erikafox Erikafox closed this Aug 5, 2023
@Zevotech
Copy link
Contributor Author

maptainers reopen pls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. Map Change Tile placing is hard. Thank you for your service.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants