-
Notifications
You must be signed in to change notification settings - Fork 66
[BOUNTY] [TEST-MERGE ONLY] Teshari #787
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
base: master
Are you sure you want to change the base?
Conversation
|
YEAAAAH!!!!!! |
| /// The tablerun ability we will give to our teshari. | ||
| var/datum/action/innate/teshari_tablerun/tablerun | ||
|
|
||
| /datum/language_holder/teshari |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
btw we will have to revisit schiechi as we develop teshari, its a 1:1 from nova
|
idk why screenshot tests are failing, i tested the clothes the moth is wearing. theyre fine |
Signed-off-by: nikothedude <59709059+nikothedude@users.noreply.github.com>
00-Steven
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not done reviewing this yet, started earlier but got tired/sick midway through
| if(is_flipped) | ||
| return FALSE | ||
| if(M.has_gravity() && M.mob_size > MOB_SIZE_SMALL && !(M.movement_type & MOVETYPES_NOT_TOUCHING_GROUND)) | ||
| if(M.has_gravity() && M.mob_size > MOB_SIZE_SMALL && !(M.movement_type & MOVETYPES_NOT_TOUCHING_GROUND) && (!isteshari(M))) // Doppler edit addition - Teshari can tablerun without breaking glass tables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we may not want this. its funny to run on a table and for it to instantly break but its also annoying and not how it was on nova. thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it shouldn't be an isteshari typecheck, ideally
either a trait and done as a multi-line edit early return before this, or the funny option with potential major consequences would be to make all teshari torso owners mob size small
| var/potential_file = bodyshape_icon_files["[shape]"] | ||
| if (icon_exists(potential_file, icon_state)) | ||
| local_worn_icon = bodyshape_icon_files["[shape]"] | ||
| if (shape != BODYSHAPE_HUMANOID) // EVERYTHING has this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i rly hate how i copy pasted this code around, but i thought about it and im blanking on how to procify it
| * | ||
| * If this returns TRUE, allows the species to access our customization. | ||
| */ | ||
| /datum/preference/proc/species_can_access_mutant_customization(species_typepath) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i heard ppl were confused why i procified this,
honestly, its probably just an artifact of my tendency to turn little distinct bits of functionality into, well, functions. this is totally not needed, especially as the implementation is only one step above straight copy and pasting the same code into each function in tails.dm, so if requested, i can revert this
|
|
||
| /datum/preference/choiced/tail_variation/init_possible_values() | ||
| return list(NO_VARIATION) + (GLOB.mutant_variations) | ||
| return list(NO_VARIATION) + (GLOB.mutant_variations) + list(TESHARI) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this works perfectly now btw
| * Determines if we're allowed to regenerate a mutant appearance/organ associated with the feature key. | ||
| * By default, returns FALSE if we cant have mutant organs. | ||
| */ | ||
| /datum/species/proc/can_regenerate_mutant_feature(feature_key) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same deal here as the other procified code
| RND_CATEGORY_CYBERNETICS + RND_SUBCATEGORY_CYBERNETICS_ADVANCED_RAPTORAL, | ||
| ) | ||
|
|
||
| /datum/design/teshari_advanced_r_arm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note there is no advanced tesh limb sprite so its the same as the default
|
A few things I forgot to mention Only tesh can use their ears, and anyone can use tesh tails and tesh can use non-tesh tails. |
|
Also, waiting on lore til it can be fullmerged |
modular_doppler/modular_species/species_types/teshari/teshari_organs.dm
Outdated
Show resolved
Hide resolved
…organs.dm Co-authored-by: kittysmooch <105110468+kittysmooch@users.noreply.github.com> Signed-off-by: nikothedude <59709059+nikothedude@users.noreply.github.com>
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
00-Steven
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
slowly working on this, posting these in piecemeal
| if(is_flipped) | ||
| return FALSE | ||
| if(M.has_gravity() && M.mob_size > MOB_SIZE_SMALL && !(M.movement_type & MOVETYPES_NOT_TOUCHING_GROUND)) | ||
| if(M.has_gravity() && M.mob_size > MOB_SIZE_SMALL && !(M.movement_type & MOVETYPES_NOT_TOUCHING_GROUND) && (!isteshari(M))) // Doppler edit addition - Teshari can tablerun without breaking glass tables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it shouldn't be an isteshari typecheck, ideally
either a trait and done as a multi-line edit early return before this, or the funny option with potential major consequences would be to make all teshari torso owners mob size small
About The Pull Request
Teshari! The loved and hated bids from nova and co.
This is mostly just a straight port. The same sprites, same mechanics, same everything. Theres only a few differences.
Design doc: https://hackmd.io/@bboHcH7iQpC5t0gFmxs9EA/Hy9xPpNRxg
Bundled in this PR are a lot of changes and tweaks, mainly around our bodyshape-bound clothing code. Included is also a robust sprite autogen system, if we ever add a species like this again.
Why It's Good For The Game
Teshari have been mentioned as a potential port species from the very onset of the server, and they have rare mentions in the lore pages, but no efforts to fully port them were made until now. This widens the biodiversity of the selection of species we offer, allowing for players to select a well-known and frequently requested species that is non-humanoid, as well as tweaking the bodyshape-bound clothing code for any future efforts to port species with atypical forms.
Testing Evidence
Changelog
🆑
add: Teshari
fix: Digitigrade females now recieve proper jumpsuit sprites
/:cl: