-
-
Notifications
You must be signed in to change notification settings - Fork 224
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
Add sound id - sound name map for each version #820
Comments
What do you mean by sound names? Do you mean the actual sound Identifier vs the Varint ID? (Since you can use either one on "modern" MC version in the packet) If you mean the Varint ID then yes it should be simple enough to do. |
https://pokechu22.github.io/Burger/1.19.2.html#sounds see fields. each sound has sound id and name (didn't know its called variant id). Also I noticed that mapping for some ids seems incorrect here |
Nevermind its just a sound_effect packet incorrectly parsed here on 1.19.2 and above |
Ah this will work well for 1.19.2 and lower. Since 1.19.3 and up you don't need the numerical ID to actually play the sound, you can use its full Identifier. But having it allows you to send sounds with smaller packets. So still good data to have. |
btw are you in the discord, as that would be a nicer why to discuss all these additions. |
hm, what do you mean? i still receive
yes |
PrismarineJS/minecraft-data-generator#17 Will have to make a schema here for the tests but this should work. Also I was thinking more of as a server you can send the text ids, forgot mojang would probably always use the numerical for builtin sounds. |
WDYM? server sends numerical ids |
The Mojang server does (atleast for sounds that are built into the game), its perfectly valid for a server to send an ID of 0 and fill in the other information of a packet instead of using the IDs. |
can be closed now (since it seems to be automated now)? |
sound ids (those are used in sound packet) are completely different between versions, however this is not a case with sound names. Anyway it’s always much easier to work with sound names, so any data related to sound ids would be really handy here
I don’t think it’s related to #77 which should contain names instead
The text was updated successfully, but these errors were encountered: