Skip to content

Fixes To Broken Crafting System - #507

Closed
aunncodes wants to merge 3 commits into
Swofty-Developments:masterfrom
aunncodes:fancystuff
Closed

Fixes To Broken Crafting System#507
aunncodes wants to merge 3 commits into
Swofty-Developments:masterfrom
aunncodes:fancystuff

Conversation

@aunncodes

Copy link
Copy Markdown
Contributor

Fixes the previously broken crafting system, with a major rework to the entirety of the recipe files.

Fixes the previously broken crafting system, with a major rework to the entirety of the recipe files.
@Swofty-Developments
Swofty-Developments self-requested a review August 5, 2025 09:43
Took 14 minutes
@Swofty-Developments

Copy link
Copy Markdown
Owner

The bugs in the crafting system definitely are fixed with this PR, though the parsing is very very slow.
image

@aunncodes

Copy link
Copy Markdown
Contributor Author

The bugs in the crafting system definitely are fixed with this PR, though the parsing is very very slow. image

strange, I'm not sure why it would've gotten slower. I guess in this case I'm making more checks to validate their craft and make sure their are no extra items. Someone else can check to see if there are any optimizations that could be made, but I'm not too sure.

@ArikSquad

Copy link
Copy Markdown
Collaborator

fix-1.diff
This might fix the performance issues if you apply this diff as a patch to ShapedRecipe

@aunncodes

Copy link
Copy Markdown
Contributor Author

Just saw this, thanks I'll test this when I'm home tommorow.

Adds the fixes from ArikSquad, should optimize it a bit.

Co-Authored-By: AriDev <75741608+ArikSquad@users.noreply.github.com>
@aunncodes

Copy link
Copy Markdown
Contributor Author

Added on the new commit, should hopefully fix some of the parsing lag. It's hard for me to tell on my end, as I have a very laggy server in general for some reason.

@aunncodes

Copy link
Copy Markdown
Contributor Author

@Swofty-Developments did you get a chance to test this?

@aunncodes

Copy link
Copy Markdown
Contributor Author

This PR will one day get merged... I believe in it

@ItzKatze

ItzKatze commented Nov 6, 2025

Copy link
Copy Markdown
Collaborator

surely

@Swofty-Developments

Copy link
Copy Markdown
Owner

I don’t have access to a PC rn and according to Git the merge has conflicts :(. If the conflicts could be resolved I would merge it.

@aunncodes

Copy link
Copy Markdown
Contributor Author

Woohoo. I saw that too, I'll see when I get a chance to fix it up since it's so old

@aunncodes aunncodes closed this Nov 21, 2025
@aunncodes

Copy link
Copy Markdown
Contributor Author

Was too oudated, will make a new one

@aunncodes
aunncodes deleted the fancystuff branch November 21, 2025 05:55
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.

4 participants