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

1.21.4 #33

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

1.21.4 #33

wants to merge 3 commits into from

Conversation

SuperGamerTron
Copy link

@SuperGamerTron SuperGamerTron commented Jan 21, 2025

The jar extractor is currently failing to generate item models for 1.21.4.

@rom1504
Copy link
Member

rom1504 commented Jan 22, 2025

What's the error ? Shouldn't we fix it ?

@SuperGamerTron
Copy link
Author

Lots of errors like this but with different items:

Error: ENOENT: no such file or directory, open 'D:\PrismarineJS\minecraft-jar-extractor\temp/1.21.4/assets/minecraft/models/item/vault.json'
    at Object.openSync (node:fs:596:3)
    at Object.readFileSync (node:fs:464:35)
    at extractModel (D:\PrismarineJS\minecraft-jar-extractor\image_names.js:232:31)
    at D:\PrismarineJS\minecraft-jar-extractor\image_names.js:255:21
    at Array.map (<anonymous>)
    at getItems (D:\PrismarineJS\minecraft-jar-extractor\image_names.js:253:42)
    at D:\PrismarineJS\minecraft-jar-extractor\image_names.js:330:5
    at D:\PrismarineJS\minecraft-jar-extractor\get_minecraft_files.js:16:7

The 1.21.4 jar is missing a lot of item models for some reason.

@rom1504
Copy link
Member

rom1504 commented Jan 22, 2025 via email

@SuperGamerTron
Copy link
Author

The folder structure has indeed been changed. Some models have been moved or added into a new folder.

@rom1504
Copy link
Member

rom1504 commented Jan 22, 2025

Ok let's add a if version more than 1.21.4 and point to the new folder in the script

@rom1504
Copy link
Member

rom1504 commented Jan 24, 2025

let's finish it ?

@SuperGamerTron
Copy link
Author

It's ready. The new folder uses a different model format, I'll make a pr in the jar extractor.

@rom1504
Copy link
Member

rom1504 commented Jan 26, 2025

please do

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.

2 participants