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

New lab palette set and nested chunks #49847

Merged
merged 3 commits into from Jul 16, 2021

Conversation

curstwist
Copy link
Contributor

@curstwist curstwist commented Jul 11, 2021

Summary

Content "New lab palette and some nested chunks."

Purpose of change

Continue adding content for modular labs.

Describe the solution

This is the palette set and a small selection of nested chunks that incorporate 3 of the palettes. I will PR more nested chunks separately to make these easier to review. Added new terrain (secret concrete door) which is listed in the palette. It will be used in future lab maps.

Describe alternatives you've considered

Include all nested chunks but the size would be difficult for reviewers. This will need to be merged before I add more mapgen files since I've created a new higher level folder to hold all the modular lab maps.

Testing

Play testing

Additional context

I'm not going to post screen shots of these nests because it's a lot of work to debug spawn and clip them all. I'll be adding a series of PR's that build on each other and hopefully can be merged without triggering missing content errors.

This will not effect old labs at all.

@ZhilkinSerg ZhilkinSerg added the Map / Mapgen Overmap, Mapgen, Map extras, Map display label Jul 11, 2021
@I-am-Erk I-am-Erk merged commit d85639a into CleverRaven:master Jul 16, 2021
pjf added a commit to pjf/Cataclysm-DDA that referenced this pull request Jul 19, 2021
* origin/master:
  New lab palette set and nested chunks (CleverRaven#49847)
  Added another Blacksmithing tool (CleverRaven#49899)
  Added cooking oil to boring sandwitch (CleverRaven#49961)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants