Skip to content

Replace IC2 Rubber tree with GT variant - #24337

Open
NeutronSelector wants to merge 20 commits into
GTNewHorizons:masterfrom
NeutronSelector:disable-ic2-tree-generation
Open

NeutronSelector wants to merge 20 commits into
GTNewHorizons:masterfrom
NeutronSelector:disable-ic2-tree-generation

Conversation

@NeutronSelector

@NeutronSelector NeutronSelector commented Apr 20, 2026

Copy link
Copy Markdown

Not merge without parent PR

@NeutronSelector NeutronSelector changed the title Disabled rubber tree world gen for GT5U replacement Replace IC2 Rubber tree with GT variant Apr 20, 2026
@UltraProdigy UltraProdigy added Ongoing Freeze - Do NOT Merge For PRs submitted close to a release that must wait for the next dev cycle to merge. Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison. labels Apr 20, 2026
@NeutronSelector
NeutronSelector marked this pull request as ready for review April 22, 2026 20:27
@UltraProdigy UltraProdigy removed the Ongoing Freeze - Do NOT Merge For PRs submitted close to a release that must wait for the next dev cycle to merge. label May 4, 2026
@YannickMG

Copy link
Copy Markdown
Contributor

Thanks for starting this effort! I believe this will need other PRs it at least the following repositories:

And it may be beneficial to include compat in these repositories:

@YannickMG

Copy link
Copy Markdown
Contributor
image

The regular pattern in-between blocks is pretty striking. It's not bad to be unique, but I don't think it's supposed to look less natural than vanilla trees right?

Also given that you're not meant to be able to tell the difference in-game, natural rubber logs should be named "Rubber Wood" instead of "Rubber Wood Natural".

I also notice that a treetap is still required for harvest but we're not including one in this PR. Should we either rethink the harvesting method or include a treetap? We could always do it as a separate PR of course but it could be neat to do it in one go.

@LazyFlesh

Copy link
Copy Markdown
Contributor

instead of having the spots be made at the moment of growth, can we mark logs with a knife so that resin bleeds from them an x amount of times? It removes a bit of rng, and makes the treetap (and a knife ig) far more useful than just growing then chopping down rubber trees

@Dream-Master

Copy link
Copy Markdown
Member

Yes more realistic rubber trees are a good idea. I remember ic2 had addon mod with a far more detailed rubber harvesting and production

@Ethryan

Ethryan commented May 27, 2026

Copy link
Copy Markdown
Contributor

A GT6 style sap collection bag that you place on a cut log?

@Dream-Master

Copy link
Copy Markdown
Member

I remember the mod called IHL tools

@NeutronSelector

Copy link
Copy Markdown
Author

Regarding the textures, I completely agree they're not suitable at this stage. As I mentioned in the parent PR, I'm leaving the sound and texture aspects to other, more competent people.

These textures are awful in my opinion, but I didn't want to waste time on that.

As for suggestions to make harvesting more realistic, no problem. Perhaps we should have a validated vision before starting any changes. I simply reused the IC2 logic without reinventing the wheel.

@YannickMG

Copy link
Copy Markdown
Contributor

I propose we keep the scope of those mechanicsl changes extremely small for now so we can just get this into GTNH ASAP. That way it's no longer a blocker for IC2 removal, which is the real value we're after.

I like the idea of giving early game tools like the knife more uses. How about for now we keep the "naturally occurring sap spawning faces" mechanic and just add the Gregtech Knife as a supported tool for collecting the sap, on top of the IC2 Treetap? That way the tree is immediately usable in the pack, and a separate PR can fix the textures and if desired the mechanics.

@NeutronSelector

Copy link
Copy Markdown
Author

I propose we keep the scope of those mechanicsl changes extremely small for now so we can just get this into GTNH ASAP. That way it's no longer a blocker for IC2 removal, which is the real value we're after.

I like the idea of giving early game tools like the knife more uses. How about for now we keep the "naturally occurring sap spawning faces" mechanic and just add the Gregtech Knife as a supported tool for collecting the sap, on top of the IC2 Treetap? That way the tree is immediately usable in the pack, and a separate PR can fix the textures and if desired the mechanics.

That was my vision, at least for the mechanics. As for the textures, if no one can create them quickly, that's certainly a possibility too. Adding the knife as a compatible tool is simple.

@NeutronSelector

Copy link
Copy Markdown
Author

Go check out the parent PR, I've revised the resin generation/harvesting gameplay to make it more realistic.

@UltraProdigy UltraProdigy added the Ongoing Freeze - Do NOT Merge For PRs submitted close to a release that must wait for the next dev cycle to merge. label Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison. Ongoing Freeze - Do NOT Merge For PRs submitted close to a release that must wait for the next dev cycle to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants