jotego / jtbin

Binary files for MiSTerFPGA, Pocket and other platforms
https://patreon.com/jotego
250 stars 71 forks source link

Street Fighter II's blanka melody is wrong. #175

Closed PepAlacant closed 3 years ago

PepAlacant commented 3 years ago

Hi Jose. I noticed strange differences in other games, but in blanka's song there is something that lacks in melody, and it sounded in other releases of jtcps1.

https://youtu.be/lZWgo2Ak9sE

maxxsun commented 3 years ago

turn on the original filter

PepAlacant commented 3 years ago

turn on the original filter

Turned on and turned off, same issue.

jotego commented 3 years ago

It seems like some instruments are one octave higher than they should. Let me look at it.

jotego commented 3 years ago

The pitch is correct. I will record the original board -I have it- and compare with it.

I have verified fully: -LFO -Phase Generator

I haven't fully verified the envelope generator yet.

PepAlacant commented 3 years ago

https://www.youtube.com/watch?v=kFLJLtlAWcA

Refering to this same sound chip. Double Dragon 2 intro theme has incorrect sounds in the percussions.

the video is not the best quality, but I wanted to find the original PCB.

Here is another video with an emulated game. Previous version of DD2 core sounded right.

https://www.youtube.com/watch?v=t4Wmw7-ycwc

jotego commented 3 years ago

This version reverts some of the changes made to match NukedYT's implementation, but I keep his LFO architecture. Please try it out.

jtcps1_20210225.rbf.zip

Compiled versions
cps1            mister  20210225    28597   PASS    880649c 2a3d89b FX68K=026efe4   JT51=29bd832    JT6295=87e6f47  JTDSP16=7ab28bd JTEEPROM=51358ae
PepAlacant commented 3 years ago

still sounds at wrong tone.

PepAlacant commented 3 years ago

A thing is sure, with your previous implementation ,jtcps1_20210204 the tones were correct both in Blanka's music, Birdie`s Music, Double Dragon II percusions. They were correct for sure. Now it has no vibrato in the main melody and the tones are at a point lower. It happens with the shot of the Willow's weapon too.

Thanks for your big efforts !

jotego commented 3 years ago

It looks like this is going to need more time. I could go back to the old version, but for that one the sound in Willow is wrong.

jtcps1_20210225.rbf.zip

PepAlacant commented 3 years ago

What piece of music in willow is wrong with this core ? I want to test it.

In the 20210225 all tones are ok and vibratoes returned into the Blanka's theme,

PepAlacant commented 3 years ago

I found a thing in the track 09H ... It doesn't have the cymbals on the core. https://vgmrips.net/packs/pack/willow-cp-system#15-dragon-cave-stage-4-2

basically the entire soundtrack lacks cymbals.

PepAlacant commented 3 years ago

It's curious, because the game starts with Cymbals, but they are getting lost. In the soundtest they are completely absents.

jotego commented 3 years ago

Tested LFO for waveforms 0-2, AM, PM, all amplitudes. It is correct. Pending checks: -Envelope generator -LFO W=3 Commit https://github.com/jotego/jt51/commit/07f56af253a67a61eab2241c6d5b331e5908fe40

jotego commented 3 years ago

I think this one may be ok.

jtcps1_20210305.rbf.zip

diff --git a/hdl/jt51_noise.v b/hdl/jt51_noise.v
index 4143a6b..e9e44bb 100644
--- a/hdl/jt51_noise.v
+++ b/hdl/jt51_noise.v
@@ -96,7 +96,7 @@ end

 // Noise mix

-assign mix_sgn = eg!=10'd0 && !out;
+assign mix_sgn =/* eg!=10'd0 ^*/ ~out;

 always @(posedge clk, posedge rst) begin
     if( rst ) begin
jotego commented 3 years ago

I think this is fixed now.