Donkey Kong: Walking Sound Hack?

Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

callanabrown wrote: Thu Nov 25, 2021 1:33 am Just tried it out! It definitely sounds better. I checked an old MAME video on YouTube and indeed it sounds sequential. I think it's still a bit off from a real board, but it's a big improvement, thanks for making this happen!!
I totally agree, it sounds alot better, thank you so much :D

However, I get some weird issues. It doesn't work to reboot in to last core. And after reboot, it freezes with a stuck Jumpman and some numbers and letters are on the monitor... please see the attached photos.

With that said, I really hope that this improvement makes it in the "official" core.

All the best!
Attachments
2.png
2.png (609.43 KiB) Viewed 10596 times
1.png
1.png (333.57 KiB) Viewed 10596 times
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

thorr wrote: Thu Nov 25, 2021 7:18 am I haven't had a chance to try this yet, but can we get this into the main core if it is indeed better?
I've added the patch so that someone else can submit it, because I left GitHub when MS acquired it. Sorg or anyone else should know what to do with the patch, so someone needs to file an issue linking to the post with the zip.

Robgus wrote: Thu Nov 25, 2021 9:25 am However, I get some weird issues. It doesn't work to reboot in to last core. And after reboot, it freezes with a stuck Jumpman and some numbers and letters are on the monitor... please see the attached photos.
I can't reproduce it. When does this happen? Right away when starting a game? Are you saying that going back to the previous core does not fix it?

Does anyone else have the same problem?
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

Brilliant!

Ok. When I first added the files as you described and booted up, it worked fine. It was when I restarted the mister (to see if it booted in to "your" core), where the issues appeared. At first the core didn't start at all. I rebooted again, in to the menu and chose the patched version and then I got the numbers and letters on the monitor. So it happened right away.

When I tried to start the previous original DK-mra, the walking sound was totally off.

So I opted to remove your patch and copy my original setup to the card - so basically back to the original DK.
User avatar
Aspie
Posts: 186
Joined: Sat Sep 26, 2020 1:07 pm
Location: Manchester
Has thanked: 20 times
Been thanked: 46 times
Contact:

Re: Donkey Kong: Walking Sound Hack?

Unread post by Aspie »

There is a .patch file, is this supposed to be added to folder too?
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

The .patch file is for submitting the changes to GitHub, it's not necessary to copy it to the MiSTer.

@Robgus I don't understand how that may be happening to you, sorry. My changes are quite aseptic and shouldn't affect anything but the wav output module. The only explanation is that there is some setting that I needed to set but failed to set before compiling the core, but I have no idea which. It apparently works for everyone else too.
mhartman
Posts: 96
Joined: Fri Jul 03, 2020 1:26 pm
Has thanked: 46 times
Been thanked: 20 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by mhartman »

Robgus wrote: Thu Nov 25, 2021 9:25 am
callanabrown wrote: Thu Nov 25, 2021 1:33 am Just tried it out! It definitely sounds better. I checked an old MAME video on YouTube and indeed it sounds sequential. I think it's still a bit off from a real board, but it's a big improvement, thanks for making this happen!!
I totally agree, it sounds alot better, thank you so much :D

However, I get some weird issues. It doesn't work to reboot in to last core. And after reboot, it freezes with a stuck Jumpman and some numbers and letters are on the monitor... please see the attached photos.

With that said, I really hope that this improvement makes it in the "official" core.

All the best!
This isn’t related to the sound hack, it’s related to the high score save I believe. Mine does this and I haven’t loaded the sound hack yet. I think you have a corrupted nvram file. Delete the nvram file for DK and you should be back in business.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

Cheers! It started after I loaded the sound hack, so maybe I did something wrong... Where do I find the nvram file, and I can still have the highscore save on right?

@pgimeno No need to say sorry. I'm really happy that you took your time to look this over!
User avatar
neurorulez
Core Developer
Posts: 7
Joined: Thu Nov 18, 2021 12:31 pm
Been thanked: 7 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by neurorulez »

pgimeno wrote: Thu Nov 25, 2021 10:21 am
thorr wrote: Thu Nov 25, 2021 7:18 am I haven't had a chance to try this yet, but can we get this into the main core if it is indeed better?
I've added the patch so that someone else can submit it, because I left GitHub when MS acquired it. Sorg or anyone else should know what to do with the patch, so someone needs to file an issue linking to the post with the zip.

Robgus wrote: Thu Nov 25, 2021 9:25 am However, I get some weird issues. It doesn't work to reboot in to last core. And after reboot, it freezes with a stuck Jumpman and some numbers and letters are on the monitor... please see the attached photos.
I can't reproduce it. When does this happen? Right away when starting a game? Are you saying that going back to the previous core does not fix it?

Does anyone else have the same problem?
I was added the patch to my personal repo, and when i tested it well, i will make a pull request in your name.
https://github.com/neurorulez/Arcade-DonkeyKong_MiSTer
Let's see if they accept the changes
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

Ohhh I just noticed, there's been one commit after the latest release, which fixes an issue with high score saving (this post gave me the hint). I was using current master, which included that commit. So yes, it's expectable that you need to delete the savefile (which you would need to do in the next update anyway). (Update, see my next post)

That explains the issue that @Robgus was having. If anyone else was saving high scores, the same applies. Thanks also to mhartman for pointing in the right direction.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

pgimeno wrote: Sat Nov 27, 2021 1:31 am Ohhh I just noticed, there's been one commit after the latest release, which fixes an issue with high score saving (this post gave me the hint). I was using current master, which included that commit. So yes, it's expectable that you need to delete the savefile (which you would need to do in the next update anyway).

That explains the issue that @Robgus was having. If anyone else was saving high scores, the same applies. Thanks also to mhartman for pointing in the right direction.
Awesome!

Sorry to bother… where do I find the save file? And, I can still have the high score save on right?

Thanks!
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

Yes, you can keep the high score saving.

The high score file should be in /media/fat/config/nvram/ as seen from the MiSTer's Linux, or just /config/nvram/ relative to the SD root, depending on how you access the SD.

And I think I misunderstood the commit log, so my previous post is inaccurate. I hate merge commits >.< Apparently the fix is in the latest release, 211025, so most probably you (@Robgus) didn't update to the latest 211025 core which is why you didn't have the problem yet and only experienced it with my patched version, which includes that change.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

pgimeno wrote: Sat Nov 27, 2021 11:17 am Yes, you can keep the high score saving.

The high score file should be in /media/fat/config/nvram/ as seen from the MiSTer's Linux, or just /config/nvram/ relative to the SD root, depending on how you access the SD.

And I think I misunderstood the commit log, so my previous post is inaccurate. I hate merge commits >.< Apparently the fix is in the latest release, 211025, so most probably you (@Robgus) didn't update to the latest 211025 core which is why you didn't have the problem yet and only experienced it with my patched version, which includes that change.
There must be something with my Mister setup... I get the same issues eventhough I deleted the save file :( (the weird numbers on the monitor and a frozen jumpman). I have the "DonkeyKong_20211025.rbf" by the way.

The autoload last core works though!

Guess I'll have to keep my fingers crossed that this makes it in the official core.
User avatar
neurorulez
Core Developer
Posts: 7
Joined: Thu Nov 18, 2021 12:31 pm
Been thanked: 7 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by neurorulez »

pgimeno wrote: Tue Nov 23, 2021 3:31 am I've made this, which I don't know if it will help.

Includes patch in `git am` format. The .MRA changes are not in the patch though.

(Edit: new version fixes an inconsistency with tabs and spaces in the patch, but the core is unchanged)
Changes merged.
https://github.com/MiSTer-devel/Arcade- ... 5681168336
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

Brilliant, happy days!

I was thinking, could it be that I have a bad rom that's causing the issue?
sardine
Posts: 23
Joined: Thu Apr 29, 2021 9:41 pm
Has thanked: 1 time
Been thanked: 2 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by sardine »

I fired up my mister tonight and performed an update_all ,,,, and,,, Donkey Kong walk sound is wrong but another port of it is ok.

to me ( after also playing a few other games ) the games are running too fast ( since I perfo4md update_all , was ok previously ), could the game speed be slightly too fast its cutting off the walk sound too soon. its as if a PAL games is running on NTSC hardware as an example.
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

neurorulez wrote: Sun Nov 28, 2021 5:47 pm Changes merged.
https://github.com/MiSTer-devel/Arcade- ... 5681168336
That's great! Thank you very much.

I want to clarify that I am NOT this guy: https://github.com/pgimeno - apparently my old username got taken by someone else.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

Am I the only one here?

I did a fresh install, ran update all and I still have the weird DK issue... :shock:

I deleted every cfg file and the nvram file. The only thing I did was change horizontal to vertical, save the config and did a reset - then the issues appeared again.

Why does saving settings in the DK menu cause this, I don't get it..?

(Edit. Highscore saving was off)
netbeui
Posts: 148
Joined: Fri Jul 31, 2020 5:04 am
Has thanked: 77 times
Been thanked: 23 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by netbeui »

I have a similar problem, the only difference is that I'm saving high scores. I ran update_all and noticed my Donkey Kong was all messed up. So I deleted the NVRAM file, played a full game and entered my high score and saved it. Played a game of Donkey Kong Jr (no issues on that core btw) and when I went back to Donkey Kong I see my recently entered high score at the top of the screen but it's not in the high score table and the attract mode screen has numbers/glitches on it. No biggie, just wanted to let you know I'm also experiencing issues.
mhartman
Posts: 96
Joined: Fri Jul 03, 2020 1:26 pm
Has thanked: 46 times
Been thanked: 20 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by mhartman »

I was having the same issue, but before the walking sound hack. It’s something else. I guessed high score save but maybe something else.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

mhartman wrote: Tue Nov 30, 2021 1:13 am I was having the same issue, but before the walking sound hack. It’s something else. I guessed high score save but maybe something else.
Interesting. I had the latest, previous version and it worked flawlessly. Went back to it yesterday.
thorr
Top Contributor
Posts: 1100
Joined: Mon Jul 06, 2020 9:37 pm
Has thanked: 537 times
Been thanked: 252 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by thorr »

Just tried this and there is some garbled text at the bottom that when you walk over it, you fall through the floor and proceed to level 2. If you try it on level 2, you fall through the floor and to the top next to the girl and die.
The walking sound is missing samples (I only have one). I used the update_all script, and it appears it doesn't get everything that is needed.
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

Hm, the samples are in the MRA, maybe install it manually? It's here: https://github.com/MiSTer-devel/Distrib ... t%201).mra
thorr
Top Contributor
Posts: 1100
Joined: Mon Jul 06, 2020 9:37 pm
Has thanked: 537 times
Been thanked: 252 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by thorr »

pgimeno wrote: Tue Nov 30, 2021 11:50 pm Hm, the samples are in the MRA, maybe install it manually? It's here: https://github.com/MiSTer-devel/Distrib ... t%201).mra
Thanks for the help. This is pretty weird. The Donkey Kong I am selecting is called "Donkey Kong". However, I downloaded the above mra and overwrote a file called "Donkey Kong (US, Set 1).mra". The mra for "Donkey Kong" is dated 4/25/2021. When I select "Donkey Kong" it works but is screwed up as previously described and is missing samples. When I select "Donkey Kong (US, Set 1)" it says it can't find an rbf file for it. It seems like the update_all script isn't working correctly, at least in my case.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

I’ve tried both a manual install and a fresh install. I didn’t work for me in either case :/
User avatar
pgimeno
Top Contributor
Posts: 669
Joined: Thu Jun 11, 2020 9:44 am
Has thanked: 246 times
Been thanked: 208 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by pgimeno »

@Robgus I'm sorry, I really have no idea what may be going on. I've had weird issues that no one else seemed to have with some cores too.

Is it possible that you have two DonkeyKong cores in the cores folder? In that case there's the possibility that the wrong one is being picked.

@thorr Yes, the `Donkey Kong.mra` file is outdated but the updater apparently does not remove old files. It is no longer in the MRA list: https://github.com/MiSTer-devel/Distrib ... in/_Arcade. It seems to have been renamed to conform to MAME naming. I have no idea why the new MRA doesn't find the core though. I always do my updates by hand, so it's possible that the updater has issues, but without a proper diagnostic of what's going on, it's hard to tell.
Robgus
Posts: 90
Joined: Sun Feb 21, 2021 3:52 pm
Has thanked: 62 times
Been thanked: 5 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Robgus »

pgimeno wrote: Wed Dec 01, 2021 11:29 am @Robgus I'm sorry, I really have no idea what may be going on. I've had weird issues that no one else seemed to have with some cores too.

Is it possible that you have two DonkeyKong cores in the cores folder? In that case there's the possibility that the wrong one is being picked.
No need to say sorry, I’m really happy you had a go at this to begin with!

That can’t be the case I believe. As I said, I got the glitches after fresh install also :/
Zeosstud
Posts: 104
Joined: Sun Jun 21, 2020 4:56 pm
Has thanked: 2 times
Been thanked: 25 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Zeosstud »

Robgus wrote: Wed Dec 01, 2021 12:07 pm
pgimeno wrote: Wed Dec 01, 2021 11:29 am @Robgus I'm sorry, I really have no idea what may be going on. I've had weird issues that no one else seemed to have with some cores too.

Is it possible that you have two DonkeyKong cores in the cores folder? In that case there's the possibility that the wrong one is being picked.
No need to say sorry, I’m really happy you had a go at this to begin with!

That can’t be the case I believe. As I said, I got the glitches after fresh install also :/
I think I have the fix for this..
make sure you delete the current nvram file
then edit the .mra and at the bottom replace what is there under </rom>
with this..

</rom>
<rom index="2"></rom>
<rom index="3" md5="none">
<part>
00 00 00 00 00 FF 00 02 00 02 00 01 00 FF 02 00
00 00 61 00 00 AA 94 76
00 00 60 B8 00 03 50 00
00 00 76 41 00 01 00 00
00 00 76 21 00 01 00 00
00 00 76 01 00 01 07 07
00 00 75 E1 00 01 06 06
00 00 75 C1 00 01 05 05
00 00 75 A1 00 01 00 00
</part>
</rom>
<rom index="4"></rom>

<nvram index="4" size="179"></nvram>

<remark></remark>

<mratimestamp>20210430005030</mratimestamp>
</misterromdescription>

This fixed the issue for me, I could 100% recreate the issue your having where odd character come up and mario falls thru them, but only when you play the game a 2nd time after a save file was created.. This hack fixed it for me.. I DID not like magically come up with this, I am not a programmer at all, I just took the nvram part of an older .mra file that worked before and added replaced what was sent down from Update_All and got lucky, I hope you have similar results

-Zeosstud
zakk4223
Posts: 270
Joined: Sun May 24, 2020 10:55 pm
Been thanked: 107 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by zakk4223 »

It looks like the PR removed one line from the MRA that is part of the high score saving system, so it's probably saving and restoring to bunk addresses.
I don't know if it was removed by accident, but it needs to be there. Restoring it should fix things
Zeosstud
Posts: 104
Joined: Sun Jun 21, 2020 4:56 pm
Has thanked: 2 times
Been thanked: 25 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by Zeosstud »

It did for me for sure.. my ears are not good enough to really appreciate the sound difference but when you mess with saving high scores, well that certainly gets my attention!! Hope this is sorted for everyone soon..

Zeosstud
thorr
Top Contributor
Posts: 1100
Joined: Mon Jul 06, 2020 9:37 pm
Has thanked: 537 times
Been thanked: 252 times

Re: Donkey Kong: Walking Sound Hack?

Unread post by thorr »

Thanks everyone, you are all amazing. :-) How do we get the update_all script to fix all this? I would rather have it automated than have a screwed up system by editing things. I am ok with deleting some files and having it re-download them if needed.
Post Reply