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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatic egg planting #6239

Closed
wants to merge 7 commits into from

Conversation

PhantomEpicness
Copy link
Contributor

@PhantomEpicness PhantomEpicness commented May 4, 2024

About the pull request

Gives the queen a verb to automatically plant eggs while on ovi . This has a cost of a slower egg laying rate. It automatically turns off if there is no more suitable space nearby.

Recommended to activate after the hive is built to avoid eggs being placed in spots you want to build stuff on.

also it doesn't auto clear hatched/destroyed eggs so xenos still have to do some roomkeeping

Explain why it's good for the game

Makes managing eggs much easier as queen. Also makes so you or a drone wont have to manually plant them and can focus on better things.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

馃啈 Totalepicness5
add: Queen can now automatically plant eggs using the 'Toggle Egg Autoplanting' ability and verb. This has a cost of laying eggs slower and consuming plasma as eggs are planted.
/:cl:

@github-actions github-actions bot added the Feature Feature coder badge label May 4, 2024
@Red-byte3D
Copy link
Contributor

Can you put this under "hive management" tab from the procs? its going to get annoying to fitler out 70000 abilities if a new one is added after every 2 weeks

@PhantomEpicness
Copy link
Contributor Author

Can you put this under "hive management" tab from the procs? its going to get annoying to fitler out 70000 abilities if a new one is added after every 2 weeks

done

@Drulikar Drulikar marked this pull request as draft May 5, 2024 03:16
PhantomEpicness and others added 2 commits May 6, 2024 07:48
duilikar review

Co-authored-by: Drathek <76988376+Drulikar@users.noreply.github.com>
@PhantomEpicness PhantomEpicness marked this pull request as ready for review May 7, 2024 00:35
code/modules/mob/living/carbon/xenomorph/XenoProcs.dm Outdated Show resolved Hide resolved

for(var/turf/turf in surroundings) // In every tile, w check if there are weeds, that we can build there and if it can be obscured
var/obj/effect/alien/weeds/weeds = locate(/obj/effect/alien/weeds) in turf
if(check_alien_construction(turf, silent = TRUE) && weeds && !check_if_can_be_obscured(turf,xeno_egg) && !istype(get_area(turf), /area/interior))
Copy link
Contributor

Choose a reason for hiding this comment

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

My intention in previous feedback was that all egg placement checks would be in the same proc, not merely the layer test. The reason for this is so there can't ever be a difference in how eggs get placed one way or the other because they both call the same proc to test if its suitable. As far as whether it gives the user an error message, that just needs to be argument given to the proc e.g. silent where autoplanting would set silent true, and otherwise normal planting wouldn't have that true so you get the messages.

code/modules/mob/living/carbon/xenomorph/castes/Queen.dm Outdated Show resolved Hide resolved
@Drulikar Drulikar marked this pull request as draft May 10, 2024 09:05
Co-authored-by: Drathek <76988376+Drulikar@users.noreply.github.com>
Copy link
Contributor

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

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label May 19, 2024
@github-actions github-actions bot closed this May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants