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

(One Last Attempt) Yu-Gi-Oh! FM | Darknite #198

Closed
wants to merge 0 commits into from
Closed

(One Last Attempt) Yu-Gi-Oh! FM | Darknite #198

wants to merge 0 commits into from

Conversation

HavenKnight93
Copy link
Contributor

@HavenKnight93 HavenKnight93 commented Mar 8, 2024

IS SONG - DO NOT DELETE THIS LINE
Game: Yu-Gi-Oh! Forbidden Memories
Song: Darknite/Nightmare
Composers: Naoko Ishii, Hiroshi Tanabe, Waichiro Ozaki
Converters: HavenKnight93
Category: bgm
Tags: Fights, Gloomy

Copy link

github-actions bot commented Mar 8, 2024

Mornin'

I've analyzed your pull request and ascertained the following information from it. This will help the verifiers handle your request faster:

Is Song Upload: No
Something needs changing: No

Copy link

github-actions bot commented Mar 8, 2024

Mornin'

I've analyzed your pull request and ascertained the following information from it. This will help the verifiers handle your request faster:

Is Song Upload: Yes
Has Binary File: Yes
Has Preview: Yes
Missing Mandatory Information: None
Headers which I don't understand: None
Is new game: No
Something needs changing: No

Here's what the output will look like:

{
    "Game": "Yu-Gi-Oh! Forbidden Memories",
    "Song": "Darknite/Nightmare",
    "Composers": "Naoko Ishii, Hiroshi Tanabe, Waichiro Ozaki",
    "Converters": "HavenKnight93",
    "Category": "bgm",
    "Tags": [
        "Fights",
        "Gloomy"
    ],
    "Verified": true,
    "Date": "Fri Mar 08 2024 21:58:19 GMT+0000 (Coordinated Universal Time)",
    "Audio": "https://github.com/theballaam96/candys-shop/raw/main/previews/Yu-Gi-Oh%20Forbidden%20Memories/DarkniteNightmare.mp3",
    "Binary": "binaries/Yu-Gi-Oh Forbidden Memories/DarkniteNightmare.bin",
    "Duration": 61.11428600000001
}

Copy link

github-actions bot commented Mar 8, 2024

Mornin'

I've analyzed your pull request and ascertained the following information from it. This will help the verifiers handle your request faster:

Is Song Upload: Yes
Has Binary File: Yes
Has Preview: Yes
Missing Mandatory Information: None
Headers which I don't understand: None
Is new game: No
Something needs changing: No

Here's what the output will look like:

{
    "Game": "Yu-Gi-Oh! Forbidden Memories",
    "Song": "Darknite/Nightmare",
    "Composers": "Naoko Ishii, Hiroshi Tanabe, Waichiro Ozaki",
    "Converters": "HavenKnight93",
    "Category": "bgm",
    "Tags": [
        "Fights",
        "Gloomy"
    ],
    "Verified": true,
    "Date": "Fri Mar 08 2024 21:59:08 GMT+0000 (Coordinated Universal Time)",
    "Audio": "https://github.com/theballaam96/candys-shop/raw/main/previews/Yu-Gi-Oh%20Forbidden%20Memories/DarkniteNightmare.mp3",
    "Binary": "binaries/Yu-Gi-Oh Forbidden Memories/DarkniteNightmare.bin",
    "Duration": 61.11428600000001
}

@HavenKnight93
Copy link
Contributor Author

This will be the final attempt of Darknite/Nightmare. I added some missing instruments and fixed velocities. Hopefully this comes close to the original, but we will go from there. If this doesn't work for all of you, then DK64LOL.

@Brian0255
Copy link
Collaborator

Honestly.. I think this is just too harsh sounding. The original song is dissonant in some parts too, but it's a little more subdued and you don't notice it really. There's also some heavy toms right panned the entire time on your port and as a result, it's rough on that ear. In the original, they appear to just be centered to avoid that issue.

I also noticed that this is on the extreme quiet end in terms of loudness, so you would want to increase the volume quite a bit.

In general I don't really have a ton of suggestions here, I think that this is just an extremely challenging song to try and make work in DK64's soundfont

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

Successfully merging this pull request may close these issues.

2 participants