Having an issue with high score saving lr mame 2003 plus

mahoneyt944

@sirhenrythe5th
what I was getting at is, since Tron uses the .nv file for it's high score and does not make a .hi file ---which high score is working, and tapper is creating a .nv file and not making a .hi file just like tron....is it reasonable to think that tappers high score will be apart of the .nv file?

Both games are Bally midway so I would think they were similar in some ways at least. If so, the .nv file is being created currently and not saving the high score within it.

If this isn't the case what is being saved in the .nv file it's making?

dankcushions

scores via nv files were/are apparently unreliable: https://highscore.mameworld.info/nvramtest.htm

grant2258

This is an easy fix thankfully tested it and it works.

here is the original code

https://github.com/libretro/mame2003-plus-libretro/blob/master/src/drivers/mcr3.c#L428-L436

notice this saves videoram instead of nvram as is described here https://github.com/libretro/mame2003-plus-libretro/blob/0134c428b75882aa474f78dbbf2c6ecde49b97b7/src/drivers/mcr3.c#L35-L39

change the nvram handler to the code (linked above) to what paste below and it will work as expected. This is something i always meant to look into but never found the time.

static NVRAM_HANDLER( mcr3 )
{
unsigned char *ram = memory_region(REGION_CPU1);

	if (read_or_write)
		mame_fwrite(file, &ram[0xe000], 0x800);
	else if (file)
		mame_fread(file, &ram[0xe000], 0x800);
}

Like i said before in this case the driver needs an update nothing more nothing less. The problem here is the nvram was pointing to the wrong place in the memory map.

mahoneyt944

@grant2258

Thank you. Is this change official or do I need to edit manually?

Could you link me the updated mame2003_plus_libretro.so to replace mine?

My editor keeps crashing when I open it :(

mitu

@mahoneyt944 said in Having an issue with high score saving lr mame 2003 plus:

Could you link me the updated mame2003_plus_libretro.so to replace mine?

You can't edit the mame2003_plus_libretro.so file directly, the source needs editing and then re-compiled to produce the new mame2003_plus_libretro.so.

mahoneyt944

@mitu

Yeah I kind of figured so when I saw the jumbled mess inside the file lol. Is anyone willing to compile this update and post it?

arcadez2003

https://github.com/libretro/mame2003-plus-libretro/pull/780

mahoneyt944

@arcadez2003

Thank you, does anyone have it compiled so I can update my system manually? Getting ready to back up my pi but wanted to get this fix in there before I do.

Riverstorm

@mahoneyt944 - You'll probably need to download from source from within the RetroPie Setup menu to get the latest commits. Basically where you installed the emulator, there should be two options, download from binary or download from source.

mahoneyt944

@Riverstorm

The change made isn't official yet to update from source. That's why I was hoping someone could compile it

grant2258

Ill provide instructions for you to compile it im assuming you have a pi3 if not change the platform to rpi2.

There is things to keep in mind one you should have mame2003-plus installed via retropie so everything is setup. The code below will compile mame for you at the bash prompt via ssh or f4 depending on your preference.

git clone https://github.com/libretro/mame2003-plus-libretro.git
cd mame2003-plus-libretro/
wget https://github.com/libretro/mame2003-plus-libretro/pull/780.patch
git am 780.patch
make -j8 platform=rpi3

copy the binary over that is made over the old one and be sure to delete the source directory when finished.

mahoneyt944

@grant2258 thank you.

Riverstorm

@mahoneyt944 - Sorry, I see there are a few pull requests pending. =/

arcadez2003

@Riverstorm said in Having an issue with high score saving lr mame 2003 plus:

@mahoneyt944 - Sorry, I see there are a few pull requests pending. =/

Yeah when i retired from active dev with regards to the MAME2003+ core i gave up my contributor status and my push and pull access
when i left the libretro organization, i just wanted to add the code as per a request from @grant2258 as a favour due to him helping
me out a fair times in that dept.

So will just need to be patient until such a time as someone does the merge.

Riverstorm

@arcadez2003 - That's not an issue at all. I appreciate the past few years of hard work you put into the core and the little things you and grant keep adding. I got so used to the commits being pushed as fast as you and grant were banging them out. I didn't stop to think that it might not be merged when Mahoney had asked. No worries at all! ;)

grant2258

@arcadez2003 this isint quite working it must be shared memory needing mapped for it to work. It only worked because i had a previous nvram there is you delete the old one and create a fresh one it doesnt work will need to look into this driver more.

mahoneyt944

@grant2258 Mine works but already had an old nvram file too.

grant2258

Never ind me Its working fine i did a copy and paste error on my port is working fine here. If you have any issues let me know. test if you like move your nvram and try making a new one. Then move your old one back after testing.

mahoneyt944

@grant2258

Thanks

mahoneyt944

This post is deleted!