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

Bodyslam missing graphics #1368

Closed
mahoneyt944 opened this issue Jun 8, 2022 · 61 comments
Closed

Bodyslam missing graphics #1368

mahoneyt944 opened this issue Jun 8, 2022 · 61 comments

Comments

@mahoneyt944
Copy link
Collaborator

mahoneyt944 commented Jun 8, 2022

Missing player tiles possibly more, working in mame2000. Looks like the spritesystem is wrong and possibly some of the other configs in the machine driver

Mame2000
bodyslam-220608-192617

Mame2003-plus
bodyslam-220608-192820

@mahoneyt944 mahoneyt944 changed the title Body slam missing graphics Bodyslam missing graphics Jun 9, 2022
@mahoneyt944
Copy link
Collaborator Author

mahoneyt944 commented Jun 9, 2022

Changing the sprite system shows that the sprites are actually there lol.
bodyslam-220609-035921

@mahoneyt944
Copy link
Collaborator Author

@MistyDreams can you test this with the new video code? Might be the issue.

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 9, 2022

@mahoneyt944 you can use the code anytime its from a newer mame just a few banking niggles i want to work out for afighters. I wouldnt recommend this in master atm though really up to you guys though

@mahoneyt944
Copy link
Collaborator Author

Just pulled it into a branch called Mist for testing.

@MistyDreams
Copy link
Contributor

works fine on mine had a look bankswitch isint working i fixed it on mine a while back MistyDreams@33be925

@mahoneyt944
Copy link
Collaborator Author

Just tried this, it didn't fix the issue, bodyslam isn't setting a obj_bank either.

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 9, 2022

work fine on my port no idea why maybe something else else has changed

@mahoneyt944
Copy link
Collaborator Author

Idk something simple I'm sure but don't see it.

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 9, 2022

I updated to the latest masteri n my branch compiled it and it works fine

@mahoneyt944
Copy link
Collaborator Author

Wtf? With no changes?

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 9, 2022

yeap fine with no changes did you verify your roms i done a clean compile as well. Doesnt take long to compile this core

Screenshot from 2022-06-10 00-25-52
?

@mahoneyt944
Copy link
Collaborator Author

mahoneyt944 commented Jun 9, 2022

It's the titlescreen where they don't work. And that weird big rope bug on the back left of the ring

@MistyDreams
Copy link
Contributor

I dont have any glitches it the title screen is blank
Screenshot from 2022-06-10 00-31-04
Screenshot from 2022-06-10 00-31-14
Screenshot from 2022-06-10 00-31-23

@mahoneyt944
Copy link
Collaborator Author

Where it says coin insert

@mahoneyt944
Copy link
Collaborator Author

See the first post

@MistyDreams
Copy link
Contributor

yea its missing here but no corruption though :)

@mahoneyt944
Copy link
Collaborator Author

Yeah there's no corruption here either, it gets the corruption effect if you change the sprite system, but by doing this shows that the tiles are there.

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 9, 2022

well there is no updateproc nevermind i seen it. leave that in the todo list think its system16a not touched that one. I do have the code in my branch not tested it yet though you can take it for a wee spin if you want. Will need to fix something up first though though

@MistyDreams
Copy link
Contributor

I was going to do this at the weekend but i done it now tested and the it fixes the issue
Screenshot from 2022-06-10 01-35-13

@MistyDreams
Copy link
Contributor

ps that copyright warning is only meant to show the first time you load a game not every time!

@mahoneyt944
Copy link
Collaborator Author

There's a core option to disable the copyright warning. Shouldn't show at all if disabled.

@MistyDreams
Copy link
Contributor

I should get this afigher issue fixed this weekend. Its most likely some kind of banking issue cause the code clamps it

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 19, 2022

im going to delete my opaopa branch soon if you want to grab the fixes for this before i do feel free. you just need to copy sys16a draw sprite and hookit it up. the macro helpers are added in src/common.h

@mahoneyt944
Copy link
Collaborator Author

I can't recall where we were with this to be honest, I'm currently in the middle of closing on a new house. Not sure when I'll have time to look back.

@MistyDreams
Copy link
Contributor

well ill leave the branch open for now till you have time no rush was just cleaning up

@mahoneyt944
Copy link
Collaborator Author

Do we want to just create the segas16a driver and video driver instead? Seems you already had the vidhrdw ported. There's not that many games anyway. Plus I'd like to slowly obsolete the system16 driver in favor of the new one. Thoughts?

@MistyDreams
Copy link
Contributor

nothing to port will just drop in just need to hook it up to drivers properly. just make a skelton segas16a.c and hook up what you need too, grab it from mame090 in historic-mame.

@mahoneyt944
Copy link
Collaborator Author

mahoneyt944 commented Jun 24, 2022

@MistyDreams ok, had a chance to look through this using the old code and the issue is with the sprite priority. Mame2000 uses variable spr_pri differently than we are in mame2003-plus. Looks to be something that wasn't quite converted properly in the change over. This branch isn't correct in the sense it messes up other priorities, but you can see how it displays the title graphics now. so it's definitely where the issue occurs and probably can be fixed with a little adjustment or possibly a change to bodyslams machine init settings. See the branch:

https://github.com/libretro/mame2003-plus-libretro/compare/Test

@MistyDreams
Copy link
Contributor

use this as a reference

https://github.com/MistyDreams/mame2003-plus-libretro/blob/942f54ef3a47f06bf8c9fd80dc8203a9c5768a3d/src/vidhrdw/system16_vidhrdw.c#L222-L352

also in afighter the boat should go under the road bridges in attract mode sequence hardly a game stopping thing though

@MistyDreams
Copy link
Contributor

Since you can drop in segas16a sprite code or just straight up moved to the segas16a code to fix any tile issues as well. I Dont understand why your trying to patch this like this, but if thats the route you want to take who am i to argue.

@mahoneyt944
Copy link
Collaborator Author

I realize the newer sprite code is probably the better solution and I'll be adding that in shortly. I was just curious to the regressions and where things went wrong so long ago. tinkering for fun.

@mahoneyt944
Copy link
Collaborator Author

Well funny enough, patching the old sprite code vs the new segas16a sprite code both have this same glitch on the title screen. When she swings, there's a bar of pixels under her arm with some issues....Not sure why. Maybe priority related?
bodyslam-220624-085124

@MistyDreams
Copy link
Contributor

what glitch is that the counter being missing ?

@mahoneyt944
Copy link
Collaborator Author

Not sure what you're mean about the counter. I'm talking about the title screen under the arm when she swings.

@mahoneyt944
Copy link
Collaborator Author

Trying mjleague with the new code, this also has minor issues (so did the old). see the lower corners, the layers aren't in line or something and you get the bars where the background shines through. Most noticable over the green grass. 1 bar above "WT" and 2 bars by the strike count.
mjleague-220624-091257

@MistyDreams
Copy link
Contributor

well youll need to tweak the priorites

@MistyDreams
Copy link
Contributor

comment out // sys16_bg_priority_value=0x0e00; in bodyslam should fix it

@mahoneyt944
Copy link
Collaborator Author

Btw, I'm not knocking the code, just documenting issues as I find them so I don't forget. Lol

@mahoneyt944
Copy link
Collaborator Author

comment out // sys16_bg_priority_value=0x0e00; in bodyslam should fix it

I'll try

@MistyDreams
Copy link
Contributor

add sys16_bg_priority_mode = 2;
too mjleage should fix that as well. Dont really like going through this old code but tweaks are needed for some things

@mahoneyt944
Copy link
Collaborator Author

Well while that does improve the title screen, it's messes up other priorities. When you get tossed out of the ring your player shows through the health bars
bodyslam-220624-094116

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 24, 2022

and this is the problem with this video system it can be fixed though youll just need to put the work in or use one that works. youll need to debug the tilemap transparency's

@MistyDreams
Copy link
Contributor

I personally think your obsession with fixing this old video code will get you grey hair but in fairness tinker away you will get there.

@mahoneyt944
Copy link
Collaborator Author

I'm trying the newer sprite code but I'm not seeing any improvement over just tweaking the old stuff in regards to bodyslam and mjleague. Maybe these things would be fixed if I used all of the newer vidhrdw from segas16a, but I just don't have the time to do it myself right now and test it properly.

@mahoneyt944
Copy link
Collaborator Author

mahoneyt944 commented Jun 24, 2022

mjleague issue is seems to be a y alignment, you can see the right side of the letters where it's split, if it were up a row of pixels it would line up correctly.
mjleague-220624-091257_1

@MistyDreams
Copy link
Contributor

MistyDreams commented Jun 24, 2022

if you go into this level of pickiness youll find faults with most games in this ole driver

@mahoneyt944
Copy link
Collaborator Author

I don't see it as picky to identify issues and see if they have easy solutions.....isn't that the whole point? Idk... I'm just going to tweak the old code to fix this issue with bodyslam and leave it open for someone else to take it further if they wish.

@mahoneyt944
Copy link
Collaborator Author

Just tried current mame, these shadows behind the health bar are present there too. But should they be? Setting fg priority mode 2 prevents the shadows. So we have to decide if we want the shadow or not?

bodyslam-220624-094116

@mahoneyt944 mahoneyt944 reopened this Jun 24, 2022
@MistyDreams
Copy link
Contributor

try fbneo if unsure its anyones guess I assumed you knew it was wrong. Maybe try find some real arcade footage

@MistyDreams
Copy link
Contributor

just compiled the latest master sprites are still missing from the title screen

@MistyDreams
Copy link
Contributor

ah never mind master was behind n mine. Im going to stop working on this segas16a till thing quite down here sick of rebasing haha

@mahoneyt944
Copy link
Collaborator Author

Oh good 😂 thought there was something else still missing lol. Besides the shadow thing to determine, this game looks good to me now. I think the only true way to know is actual arcade footage but who knows if that's out in the wild or not.

The only other thing I noticed was, in current mame, there's a "curtain raise" effect before the start of the round. Ours just goes right to the match. The effect doesn't look natural to me so I need to see if this effect is suppose to happen or not. I'm guessing it's a y scroll hookup. If it's supposed to do it.

@mahoneyt944
Copy link
Collaborator Author

Just tried fbn and there's no shadows, while current mame has shadows. So it's anyone's guess. I think no shadows seems right but idk, no need to worry about that one.

For the curtain thing we are missing it. Probably the yscroll hookup. Here's a pic in the middle of it raising.
bodyslam-220625-001035

@MistyDreams
Copy link
Contributor

its probably the 8751 protection needing done proper like the counter screen being missing too quartet does the same

@MistyDreams
Copy link
Contributor

there also priority issues i afighter you might want to look at if you want to fix things, I personal dont feel they are game breaking anyway

@MistyDreams
Copy link
Contributor

its also might be worth closing threads for each game and make one system16 bugs thread people can just add to the list that way.

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