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

Prefab - Gnomish Warship #5987

Closed
wants to merge 46 commits into from
Closed

Conversation

vocalpocal
Copy link
Contributor

@vocalpocal vocalpocal commented Sep 12, 2021

About the PR

This is a contest entry for https://forum.ss13.co/showthread.php?tid=16992
Summary:

  • Size: 13x19
  • Probability: 25
  • Enemies: 2 (Maneaters)
  • Merchant: David the Gnome
  • Other: Two garden gnomes which water plants
  • Equipment: Three hydroponic trays which spawn with strange seeds in them - otherwise regular trays, four arable soil trays, and some green clothes

The gnomish warship

image

The cockpit is fiercely defended by two very hungry maneaters. The engine room has been renovated into a laboratory for alien plant research. The ship is manned by two gnome critters, who keep the garden pretty. In the cockpit you can find the ship's merchant captain, David the Space Gnome!

David the Space Gnome

image
image

David sells many useful botany supplies, including fertilizers and strange seeds. He is quite stingy, so his wares don't come cheap! He himself buys honey (because it is tasty), human and monkey meat (to feed the maneaters), but not synth meat.

David the Spacegnome has two secret items, but he wont sell them to just anyone! In order to unlock his most valuable secret shop items, you have to show him a very high-class botany produce. He is not interested in herbs or other crap you might produce, he wants to see a quality snack!

Gnome critters

image

These helpful fellows water plants when they need it most. They also clear trays from dead plants, and very rarely plant seeds to empty hydroponic trays. Huggable. Unfortunately, they are mute.

Why's this needed?

This is a contest entry. Gnomes are cute.
The merchant sells many useful (yet expensive) tools for botanists to counteract some of the unintended side effects of piss nerfing -- botany actually suffered a lot from it and countless strats were invalidated when saltpetre output dropped by over 90%. You can get saltpetre through other means (such as QM, fabricating more GardenGears) but emptying dozens of tiny bottles into bigger containers makes it very tiresome. David sells saltpetre in larger bottles, making the process a bit easier (if you have the money).
The 50000 credit splicing module enables very interesting strategies which have not been feasible before. When installed to your Plantmaster Mk3, it improves its minimum splice chance from 0% to 30%. If you somehow get past the maneaters, and have the money for the module, then you have almost unlimited potential in your splices --- but failures are still possible.

Changelog

(u)Vocalpocal
(*)Added a gnomish warship prefab. Go say hi to David the Space Gnome, who sells hydroponics supplies!

@vocalpocal vocalpocal requested a review from a user September 12, 2021 15:19
@boring-cyborg boring-cyborg bot added A-Mapping A mapping change C-Sound Automatically applied on any .ogg or sound folder change. C-Sprites Automatically applied on any .dmi or icons folder change labels Sep 12, 2021
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 12, 2021
@keywordlabeler keywordlabeler bot added C-Feature A new feature or enhancements to existing features E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. labels Sep 12, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2021

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 8, 2021
Copy link
Contributor

@flrsh flrsh 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 the theme and I think it'd be a unique prefab, see my comments for some considerations around the trader's inventory.

code/datums/commodity.dm Outdated Show resolved Hide resolved
code/datums/commodity.dm Outdated Show resolved Hide resolved
code/datums/commodity.dm Outdated Show resolved Hide resolved
comtype = /obj/item/reagent_containers/food/drinks/drinkingglass/pitcher/pisscher
desc = "Botanist's friend and precursor for a great fertilizer... but why is it in a pitcher!?"
onmarket = 0
price = 1000
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably more like 2000, since you can make 360u saltpetre.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remember that this is one of those items which you can't buy from David before you have completed his mini-quest, and you used to be able to do something similar in less than two minutes.

Comment on lines +556 to +563
/datum/commodity/splicer
comname = "LUCA 1005 - Splicer Upgrade"
comtype = /obj/item/plantModule/splicer
desc = "Increases PlantMaster Mk3 splicing chances by using the science behind the genome of the last universal common ancestor."
onmarket = 0
price = 50000
upperfluc = 10000
lowerfluc = -10000
Copy link
Contributor

Choose a reason for hiding this comment

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

Unsure about this, @Tarmunora thoughts?

else
if (--plant_check < 1)
plant_check = initial(plant_check)
for (var/obj/machinery/plantpot/planter in view(7, src))
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
for (var/obj/machinery/plantpot/planter in view(7, src))
for (var/obj/machinery/plantpot/planter in view(3, src))

Should be significantly smaller.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's the same as for bees, because otherwise they might wander too far. What if I made them check for trays less frequently?

var/obj/machinery/plantpot/work_target
var/work_bore = 40

ai_think()
Copy link
Contributor

Choose a reason for hiding this comment

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

You should have them hibernate or go to sleep if there are no players around; otherwise they'll just keep processing and checking their surroundings, adding extra unnecessary load.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fair. I did not know sleep did that and thought it was just a visual thing. But also on the other hand trays might dry in the prefab if gnomes are not watering them... not like it really matters but it would be visually more pleasing I guess.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wait, I think the critter parent already does this on its own, unless I am completely mistaken.

@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 10, 2021
@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Oct 12, 2021
@TTerc TTerc removed the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 12, 2021
@github-actions github-actions bot added S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict and removed S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict labels Oct 13, 2021
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 28, 2021
@vocalpocal
Copy link
Contributor Author

oh right this was still up
uh I will fix the AI soonish

@github-actions github-actions bot removed the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 29, 2021
@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Nov 8, 2021
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Nov 23, 2021
@TTerc TTerc removed the S-Stale An inactive PR that has had no updates in the past two weeks label Nov 23, 2021
@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Dec 1, 2021
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Dec 16, 2021
@Tarmunora Tarmunora closed this Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Mapping A mapping change C-Feature A new feature or enhancements to existing features C-Sprites Automatically applied on any .dmi or icons folder change E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict S-Stale An inactive PR that has had no updates in the past two weeks size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants