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

YM2612 Nuked Driver Implementation possible? #1682

Open
Posey999 opened this issue Feb 18, 2024 · 12 comments
Open

YM2612 Nuked Driver Implementation possible? #1682

Posey999 opened this issue Feb 18, 2024 · 12 comments

Comments

@Posey999
Copy link

Posey999 commented Feb 18, 2024

Rules and recommendations

  • don't request support for currently unsupported hardware (arcade boards, consoles and computers)
  • try to provide as much information as possible (screenshot, savestate, platform, settings, ...)
  • make sure it happens with our latest official build from https://github.com/finalburnneo/FBNeo/releases/tag/latest
  • any issue on derivated projects that doesn't happen with our latest official build must be reported to that derivated project's maintainers instead

so, the current Mame ym2612 sound Driver Implementation is Out of Date and isn't anywhere near as good as GenGXPlus's Modern Nuked Driver, i am wondering if someone would be so kind as to express their technical code wizardry and add Nuked Support for Yamaha OPN2 Emulation

on another note, Nuked YM2612 mode plus a Low pass Filter of 55% is about 99.9% accurate to the sound of a Real Model 1 Genesis, and it capable of correctly playing back Metal Squad from Thunder Force 4

Nuked Driver Implementation in Gen Plus GX
(https://github.com/nukeykt/Genesis-Plus-GX)

Nuked Driver Source Code
(https://github.com/nukeykt/Nuked-OPN2)

@Posey999
Copy link
Author

bandicam.2024-02-18.10-50-37-524.mp4

this is the current implementation of FB Neo's Mame driver on the latest as of today

@Posey999
Copy link
Author

https://youtu.be/ylq90xrdDrI?t=169

here is Real Hardware using a Timestamp

@Posey999
Copy link
Author

Posey999 commented Feb 18, 2024

Untitled_Project_V1.mp4

here is Genesis GX Plus using the Nuked Driver with a 55% Low Pass filter
any discrepancies found in the video are not the Nuked Drivers Fault, it is an Older Build of Genesis GX Plus, with a Z80 Timing bug, will test latest version, and then update

@Posey999
Copy link
Author

here is an updated version of the audio

retroarch.2024-02-18.11-20-07-659.mp4

@dinkc64
Copy link
Collaborator

dinkc64 commented Feb 19, 2024

yes, its a great idea

@dinkc64
Copy link
Collaborator

dinkc64 commented Feb 28, 2024

well, looking into this, I really don't know what to port, maybe you can help?
the nuked driver in gensplusgx seems a bit different than just the nuked driver?
what one do you want - what one is the best/most preferred and why?

best regards,

  • dink

@barbudreadmon
Copy link
Collaborator

Looks like the version of ym3438.c in the genplusgx repo is a tad older than the one in the library repo (header says 1.0.8 vs 1.0.12). I suppose the newer version is the preferred one ?
Maybe we should ask more information from its author @nukeykt ?

@dinkc64
Copy link
Collaborator

dinkc64 commented Feb 28, 2024

yes, good idea :)

@dinkc64
Copy link
Collaborator

dinkc64 commented Feb 28, 2024

Hi @nukeykt,
We want to add your nice & impressive FM core to our MD emu, but not sure
which code we should port over. Should we use the code from:

https://github.com/nukeykt/Genesis-Plus-GX
or
https://github.com/nukeykt/Nuked-OPN2 ?

thanks & best regards,

  • dink

@nukeykt
Copy link

nukeykt commented Feb 29, 2024

use standaone repostory (https://github.com/nukeykt/Nuked-OPN2), it is always up to date.
also, GPGX seems to use latest version? (https://github.com/ekeeke/Genesis-Plus-GX/blob/master/core/sound/ym3438.c)

@rtissera
Copy link

This would definitely be a great addition 👍

@dinkc64
Copy link
Collaborator

dinkc64 commented Jun 25, 2024

I didn't forget, it's just been a very busy time for me!

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

5 participants