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

Add sound id - sound name map for each version #820

Open
zardoy opened this issue Jan 4, 2024 · 10 comments
Open

Add sound id - sound name map for each version #820

zardoy opened this issue Jan 4, 2024 · 10 comments

Comments

@zardoy
Copy link

zardoy commented Jan 4, 2024

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

@wgaylord
Copy link
Contributor

wgaylord commented Jan 9, 2024

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.

@zardoy
Copy link
Author

zardoy commented Jan 9, 2024

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

@zardoy
Copy link
Author

zardoy commented Jan 10, 2024

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

@wgaylord
Copy link
Contributor

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.

@wgaylord
Copy link
Contributor

btw are you in the discord, as that would be a nicer why to discuss all these additions.

@zardoy
Copy link
Author

zardoy commented Jan 10, 2024

Since 1.19.3 and up you don't need the numerical ID to actually play the sound

hm, what do you mean? i still receive hardcodedSoundEffectHeard with numerical sound ID from most events of the world (sound to play) on 1.19.2 and above

btw are you in the discord, as that would be a nicer why to discuss all these additions.

yes

@wgaylord
Copy link
Contributor

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.

@zardoy
Copy link
Author

zardoy commented Jan 12, 2024

Also I was thinking more of as a server you can send the text ids

WDYM? server sends numerical ids

@wgaylord
Copy link
Contributor

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.

@zardoy
Copy link
Author

zardoy commented Jul 13, 2024

can be closed now (since it seems to be automated now)?

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

No branches or pull requests

2 participants