Skip to content

mapped sounds from 26.1 back through 1.19.4#102

Merged
kennytv merged 9 commits intoViaVersion:mainfrom
mfishma:main
Mar 14, 2026
Merged

mapped sounds from 26.1 back through 1.19.4#102
kennytv merged 9 commits intoViaVersion:mainfrom
mfishma:main

Conversation

@mfishma
Copy link
Contributor

@mfishma mfishma commented Mar 11, 2026

Using https://pokechu22.github.io/BurgerWebViewer to sanity-check the existence of sounds in each version, I mapped most items, leaving out ones that didn't make sense, and adhering to the mapping, in that, a fall sound stays a fall sound, a break sound maps to a break sound, etc.

Of note:

  • In 26.1, I switched the "harp" notes for the new trumpet instrument with didgeridoo.
  • In mapping-1.20to1.19.4.json, I removed what looked like 2 duplicate entries.

mfishma added 2 commits March 11, 2026 00:23
used https://pokechu22.github.io/BurgerWebViewer to sanity check existence of sounds in each version, but this still needs to be tested.
@kennytv
Copy link
Member

kennytv commented Mar 11, 2026

See the failing test

parrot.imitate.camel_husk fixed
Switched hurt and death sounds of ghast family and nautiluseses.
@mfishma mfishma requested a review from kennytv March 13, 2026 01:45
@mfishma
Copy link
Contributor Author

mfishma commented Mar 13, 2026

Ready for re-review

@kennytv
Copy link
Member

kennytv commented Mar 13, 2026

+ still failing tests

@mfishma
Copy link
Contributor Author

mfishma commented Mar 13, 2026

Ready for re-review, I resolved errors running the tester.
Just need to confirm if your previous comments were referring to "entity.ghastling.spawn": "entity.ghast.scream" (which isn’t really as intense as the hurt sounds), and if so, we could go with another ghast.ambient mapping or block.chorus_flower.grow.

@kennytv
Copy link
Member

kennytv commented Mar 13, 2026

The scream is a bit too much as well imo yeah, less so than hurt, but still

@mfishma
Copy link
Contributor Author

mfishma commented Mar 14, 2026

OK, updated that one to ghast.ambient. Please take a look at your convenience.

Copy link
Member

@kennytv kennytv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot ! This'll be merged into 5.8.0

@kennytv kennytv merged commit 848b439 into ViaVersion:main Mar 14, 2026
1 check passed
@mfishma
Copy link
Contributor Author

mfishma commented Mar 14, 2026

Oh yeah, linking to ViaVersion/ViaBackwards/issues/326

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