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

Edit fracturerift to have uneven walls #1

Merged
merged 3 commits into from
Mar 3, 2015

Conversation

HeroOfTheWinds
Copy link
Contributor

I tried my hand at making uneven walls to the chasm, but it still needs work. Not sure how to proceed, though. Ignore the "Merge pull request...", I couldn't figure out how to get git not to do that.

HeroOfTheWinds and others added 2 commits March 1, 2015 12:36
oops, accidentally turned off ore thinning!
Added code to make the walls of the rift uneven... some of the time.
Still WIP, but best results so far are here.  Removed code for finite
bottom, as it is no longer used.  Commented out lines 53-56 to enable
wall varying.
@HeroOfTheWinds
Copy link
Contributor Author

Line 9 wasn't commented out in the pull request, so there's no error. :)

A little more complicated, but far better results.  Changed
fracrift_material to sandstone, since that looks more natural by water.
Explained added code better.  Uncommented vital line.
@HeroOfTheWinds
Copy link
Contributor Author

Actually, that was an error, so I fixed it, and came up with a much better (and prettier) solution to chasm variation. Not planning any more work on this section any time soon, barring major errors.

Kilarin added a commit that referenced this pull request Mar 3, 2015
Edit fracturerift to have uneven walls
@Kilarin Kilarin merged commit 496ea93 into Kilarin:master Mar 3, 2015
@Kilarin
Copy link
Owner

Kilarin commented Mar 3, 2015

Simply incredible! THANK YOU

Kilarin pushed a commit that referenced this pull request Mar 5, 2015
Merge pull request #1 from HeroOfTheWinds/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants