Skip to content

Add missing vines and thorns for crimson/corruption blocks sets #42

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

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Conversation

vanaigr
Copy link
Contributor

@vanaigr vanaigr commented Feb 18, 2023

Originally both crimson and corruption blocks sets only contained vines and not thorns, and "Corrupted Vines" block id was that of corruption thorns, so crimson and corruption blocks sets actually contained different sets of blocks.

And if crimson/corruption blocks sets were intended for searching the blocks needed to be removed in order to make the world pure, the sets were incomplete as both crimson/corruption thorns and (probably) vines all count as blocks of their respective biomes and affect the world purity, and neither set contained both vines and thorns.

The fact that corruptions blocks set was missing corrupt vines (and instead contained corruption thorns) is probably not a big issue, as crimson/corruption vines can only grow on the grass blocks of the corresponding biome. But crimson blocks set not containing crimtane thorns is actually a problem because the thorns can be located anywhere and are hard to spot, and just a few (maybe even one) thorn block is enough to render the world as being 1% crimson.

Crimson Blocks set was missing crimson thorns and Corruption Blocks were missing corrupt vines
@Mochnant
Copy link

Mochnant commented Oct 9, 2023

As a user who is stuck at 1% with no blocks shown in TerraMap using the Crimson set, this would be very helpful!

@jasoncoon jasoncoon merged commit ad8b046 into TerraMap:master Oct 9, 2023
@jasoncoon
Copy link
Contributor

Sorry for the delay in merging this, thanks for the pull request!

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.

3 participants