Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@Chubbygummibear
Copy link
Contributor

@Chubbygummibear Chubbygummibear commented Mar 6, 2023

previously #14861

Round 3?

yeah i'm like 90% 70% certain i ironed out the preference savefile quirks and it won't break your characters into grayscale monstrosities

Your prefs will auto save with the full hexcode colors while retaining your color choices next time you launch and this is active

Document the changes in your pull request

More specifically ports

What does this actually do?

well if i did everything perfectly right, 90% of players won't notice anything changed. xeno traits such as lizard frills, pod hair, and moth wings are all stored in a separate chunk of dna that everyone has, meaning they can be edited through genetics as well as dna injectors working on the same species instead of only working for and on humans.

Did you spend a week month year porting tg's dna code just to add stuff to genetics?

yeah

Wiki Documentation

I'm unsure how much the wiki covers about dna, but maybe a blurb about DNA storing mutant features now, and genetics being able to alter these

Changelog

🆑 @Y0SH1M4S73R, @Ghommie, Chubbygummibear
rscadd: The DNA console can now edit the nonhuman features of its occupants, as well as allowing injectors to apply those features. Look for the "Features" tab in the DNA console.
rscadd: everyone has a whole new dna block called unique_features that holds things like mutant color and alien bits settings. unique features can be altered and copied onto other people through genetics. this only works on people of the same species
wip: this is probably going to need a lot of ongoing work
tweak: hexcolors are handled differently
bugfix: glowy color works
bugfix: extendo arm color matches your mcolor
tweak: removes color_legacy prefs and changes sanitize_hexcolor()'s default to 6 characters
experimental: Whole lotta DNA and color code reworking
/:cl:

Chubbygummibear and others added 12 commits July 13, 2022 23:49
* ho

* pod nerfed by a pod main

* sugone

* yea

* yer

* lotsa stuff

* tgui is pain

* i don't know

* maybe?

* god kill me

* tailed

* wait no i fixed it wrong, this should be better

* SCOPE CREEP BABEEEEEEEEEE

* sins

* mother of all omlettes jack

* is it over?

* ethereal colors because idk

* i think that's it until we find out what else i broke

cl ass

clean up debug messages

last one probably

* Revert "i think that's it until we find out what else i broke"

This reverts commit 574b0e8.

* last time with gusto

* hhh

* ho

* pod nerfed by a pod main

* yer

* fix

* Revert "fix"

This reverts commit 979eddb.

* Delete plantpeople.dm

* Revert "Merge branch 'dna-blocks-but-like-good' of https://github.com/Chubbygummibear/Yogstation-TG into dna-blocks-but-like-good"

This reverts commit 1756ff2, reversing
changes made to 05f88be.

* this shouldn't be in here

* missed that

* so many little things

* please for the love of god work
@Chubbygummibear Chubbygummibear requested a review from a team as a code owner March 6, 2023 08:59
@github-actions github-actions bot added the Admin This PR affects administrators label Mar 6, 2023
@MajManatee
Copy link
Contributor

The manmade horrors beyond our comprehension

Copy link
Contributor

@Moltijoe Moltijoe left a comment

Choose a reason for hiding this comment

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

a lot of this is just changing variable names and extending colours into proper hex codes

also, color conflicts

Copy link
Contributor

@Moltijoe Moltijoe left a comment

Choose a reason for hiding this comment

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

i can't find anything wrong, most of this is just rearranging code and changing hexvalues
the new parts don't look broken, so hopefully there isn't a secret bug mixed in
perhaps overly hopeful considering this game

@Moltijoe Moltijoe added Awaiting - Action - HD Awaiting action from Jamie Merge conflict Files are conflicting with current live changes. labels May 2, 2023
@JohnFulpWillard
Copy link
Contributor

would be awesome to see this become real

@Bibby0110
Copy link
Contributor

Damn this is stale

@Bibby0110 Bibby0110 closed this Jul 29, 2023
@Yogbot-13 Yogbot-13 added Feature This adds new content to the game Fix This fixes an issue. Please link issues in fix PRs Tweak This PR contains changes to the game. labels Jul 29, 2023
@SapphicOverload
Copy link
Contributor

I'll reopen this if the merge conflicts get fixed

* ho

* pod nerfed by a pod main

* sugone

* yea

* yer

* lotsa stuff

* tgui is pain

* i don't know

* maybe?

* god kill me

* tailed

* wait no i fixed it wrong, this should be better

* SCOPE CREEP BABEEEEEEEEEE

* sins

* mother of all omlettes jack

* is it over?

* ethereal colors because idk

* i think that's it until we find out what else i broke

cl ass

clean up debug messages

last one probably

* Revert "i think that's it until we find out what else i broke"

This reverts commit 574b0e8.

* last time with gusto

* hhh

* ho

* pod nerfed by a pod main

* yer

* fix

* Revert "fix"

This reverts commit 979eddb.

* Delete plantpeople.dm

* Revert "Merge branch 'dna-blocks-but-like-good' of https://github.com/Chubbygummibear/Yogstation-TG into dna-blocks-but-like-good"

This reverts commit 1756ff2, reversing
changes made to 05f88be.

* this shouldn't be in here

* missed that

* so many little things

* please for the love of god work
@Chubbygummibear
Copy link
Contributor Author

merge conflicts resolved. again. for the morbillionth time. if i see merge conflicts again i would rather delete this entire branch than maintain it again

Yogbot-13 added a commit that referenced this pull request Jul 31, 2023
Yogbot-13 added a commit that referenced this pull request Aug 1, 2023
Yogbot-13 added a commit that referenced this pull request Aug 1, 2023
Copy link
Contributor

@ToasterBiome ToasterBiome left a comment

Choose a reason for hiding this comment

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

looks good

@ToasterBiome ToasterBiome merged commit a797efa into yogstation13:master Aug 1, 2023
Yogbot-13 added a commit that referenced this pull request Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Admin This PR affects administrators Awaiting - Action - HD Awaiting action from Jamie Feature This adds new content to the game Fix This fixes an issue. Please link issues in fix PRs Merge conflict Files are conflicting with current live changes. Tweak This PR contains changes to the game.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants