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

Add a dive shop location #55535

Merged
merged 4 commits into from
Feb 21, 2022
Merged

Add a dive shop location #55535

merged 4 commits into from
Feb 21, 2022

Conversation

ccaviness
Copy link
Contributor

@ccaviness ccaviness commented Feb 20, 2022

Summary

Content "Add a dive shop location"

Purpose of change

Add a new dive shop location to provide another location for sourcing
neoprene and other SCUBA related items.

Describe the solution

Add a dive shop with two variations. One has an open class area and a
small pool in the back, as well as a spot for refilling SCUBA tanks, and
a small mob of pool monsters. The other has a larger separate
classroom and more sales floor space.

Describe alternatives you've considered

Not adding content.

Testing

Spawned using overmap editor, and examined the building. Also verified
the dive shop is showing up in cities at a reasonable rate.

Additional context

Dive shop with classroom:
dive_shop_with_classrom
Dive shop with pool:
dive_shop_with_pool

@ccaviness
Copy link
Contributor Author

This is the first bit of content I've tried creating, so please let me know if I've done something incorrectly. I started out by copying the antique store and making some changes. I don't know if my edits to data/json/overmap/multitile_city_buildings.json are correct; I made them to match what I found when looking for antique store references.

Add a dive ship with two variations. One has an open class area and a
small pool in the back, as well as a spot for refilling SCUBA tanks.
The other has a larger separate classroom and more sales floor space.

This will provide another location for finding nepoprene and other SCUBA
related items.
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Feb 20, 2022
@Night-Pryanik
Copy link
Contributor

Overall looking good.

Nitpicks:

  1. Classroom variant could use a toilet. You know, sellers are people with needs too.
  2. Wardrobes in a shop look slightly out of place. Wardrobe is an opaque wooden cabinet more fitting for a house rather than a shop. Maybe replace them with locker or glass front cabinet?
  3. weight field is useful only if you plan to make variants differ by rarity. If they all have the same chance to spawn, this field is redundant. It's not an error though.
  4. Consider adding floor plants or vending machines.
  5. Consider replacing wooden t_floor tiles around the pool with something more water-resistant.
  6. Consider adding books on swimming in classroom variation.
  7. Consider adding windows in a classroom.
  8. Consider adding air condition units.

@ccaviness ccaviness marked this pull request as draft February 20, 2022 19:46
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Feb 20, 2022
@ccaviness ccaviness marked this pull request as ready for review February 20, 2022 22:06
@ccaviness
Copy link
Contributor Author

Thanks for the review - definitely appreciate the tips.

  1. Classroom variant could use a toilet. You know, sellers are people with needs too.

Done.

  1. Wardrobes in a shop look slightly out of place. Wardrobe is an opaque wooden cabinet more fitting for a house rather than a shop. Maybe replace them with locker or glass front cabinet?

Went with the glass cabinets.

  1. weight field is useful only if you plan to make variants differ by rarity. If they all have the same chance to spawn, this field is redundant. It's not an error though.

Adjusted - I mean for the pool variant to be less common, but during testing I set the weights the same so I could more reliably spawn both.

  1. Consider adding floor plants or vending machines.

Added a couple of both here and there.

  1. Consider replacing wooden t_floor tiles around the pool with something more water-resistant.

Concrete.

  1. Consider adding books on swimming in classroom variation.

Done - added swimming and some first aid books.

  1. Consider adding windows in a classroom.

Added a few.

  1. Consider adding air condition units.

Tweaked the roof more - added a skylight over the pool/classroom area, a couple of solar panels and an AC unit., and some standing tanks.

Updated the screenshots in the PR description.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 20, 2022
@Maleclypse Maleclypse added Map / Mapgen Overmap, Mapgen, Map extras, Map display Spawn Creatures, items, vehicles, locations appearing on map [JSON] Changes (can be) made in JSON labels Feb 21, 2022
@ZhilkinSerg ZhilkinSerg merged commit 6ca3ef2 into CleverRaven:master Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants