Skip to content

New MIDI for E1M5 #730

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

Merged
merged 2 commits into from
Jul 26, 2021
Merged

New MIDI for E1M5 #730

merged 2 commits into from
Jul 26, 2021

Conversation

northivanastan
Copy link
Contributor

I always found it kind of annoying that this same MIDI was used four times in Phase 1; D_E1M5 and D_E2M6 are identical and both are reused in E4. So here's a new D_E1M5. The E1M5 map feels dark and empty to me, despite the relatively bright lighting in places, so I decided that my MIDI should emphasize that emptiness.

Please do provide any feedback!

@northivanastan
Copy link
Contributor Author

Looks like this would solve issue #499, someone else apparently had a MIDI for this but it was lost.

@northivanastan
Copy link
Contributor Author

Also released the MIDI on OpenGameArt, for easier access.

@ghost
Copy link

ghost commented Apr 10, 2021

I really like this one, I fully support this being the track for E1M5.

@northivanastan
Copy link
Contributor Author

northivanastan commented Apr 11, 2021

Someone on Doomworld pointed out a technical issue with this MIDI - the drumset track (used for an echoing cymbal, which you can hear on the OGA exported versions) is set to use MIDI InputPort and OutputPort 2, which will try to use a second MIDI driver, preventing some users from hearing it if they have only one MIDI driver. Help from MIDI wizards on this would be appreciated!

@northivanastan
Copy link
Contributor Author

The same person on Doomworld (CammyBanana) fixed the MIDI, so thanks to them!

@Xindage
Copy link
Contributor

Xindage commented Jul 26, 2021

Approved these changes.

@Xindage Xindage merged commit c87f623 into freedoom:master Jul 26, 2021
@Xindage Xindage mentioned this pull request Jun 26, 2022
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