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

APU enhancement #79

Closed
wants to merge 24 commits into from
Closed

Conversation

fjpolo
Copy link
Contributor

@fjpolo fjpolo commented May 27, 2024

[APU]

  • Improved R2A03 APU’s triangle wave and made its mixer linear.
  • Improved mapper VRC6’s sawtooth wave. Mixer is already linear.

[VRC6]

  • Added mapper. Tested on Castlevania III (Japan). Needs more testing.

[Mapper4/69]

  • Disabled APU Enhancement because volume levels saturate

References:

@fjpolo
Copy link
Contributor Author

fjpolo commented May 27, 2024

APU sounds broken in Little Sisyphus.nes if Enhancements are disabled, also with Celeste Mario.nes

Edit: found a few bugs that I need to fix

@fjpolo fjpolo marked this pull request as draft May 27, 2024 17:38
@nand2mario
Copy link
Owner

Can you remove the firmware here and add a separate PR for the firmware in the snestang repo? Let's keep the firmware in a single repo (snestang) for now and make sure the same firmware works for both nestang and snestang. CORE_ID can be used to determine which core is running. In the future we can set up a separate repo for the firmware.

@fjpolo
Copy link
Contributor Author

fjpolo commented May 28, 2024

Can you remove the firmware here and add a separate PR for the firmware in the snestang repo? Let's keep the firmware in a single repo (snestang) for now and make sure the same firmware works for both nestang and snestang. CORE_ID can be used to determine which core is running. In the future we can set up a separate repo for the firmware.

Will do, this way it will be cleaner. Adding a separate repo has the advantage of adding it as a submodule.

Edit: if you create an empty repo for the firmware, I can fork it, create a separate branch to add firmware/ as submodule and create a separate PR for it.

@nikitalita
Copy link
Contributor

Does this implement the suggestions from this video? https://www.youtube.com/watch?v=8RrQrATnXXY&t=500s

@fjpolo fjpolo changed the title APU enhancement plus other things APU enhancement May 28, 2024
@fjpolo
Copy link
Contributor Author

fjpolo commented May 28, 2024

Does this implement the suggestions from this video? https://www.youtube.com/watch?v=8RrQrATnXXY&t=500s

Oh yes, that's the idea! I watched that video a long time ago and yesterday came back to my mind.

I'm still on it though. I missunderstood how the module worked so I made some mistakes.

Will change the draft to a proper PR after fixing, testing, and removing firmware/ dir to have aclean PR

- Remove unnecessary files
@fjpolo fjpolo marked this pull request as ready for review June 23, 2024 14:07
@fjpolo
Copy link
Contributor Author

fjpolo commented Jun 23, 2024

Results are sadly not as good as I'd like :( but at least enabling APU Enhancements gives the user a linear mixer, for those who'd prefer it :)

Edit: Slept on it and I think it's not ready yet. Will keep tinkering

@fjpolo fjpolo marked this pull request as draft June 24, 2024 09:32
@fjpolo fjpolo marked this pull request as ready for review June 24, 2024 12:54
@fjpolo
Copy link
Contributor Author

fjpolo commented Jun 24, 2024

After shifting Noise Channel output from 6bit to 8bit, it soudns better, at least for me.

- Noise Channel needs to be shifted left for linear mixing
- Fix VRC6 Triangle Channel
@fjpolo fjpolo marked this pull request as draft June 24, 2024 16:24
@fjpolo fjpolo marked this pull request as ready for review June 25, 2024 14:41
@fjpolo
Copy link
Contributor Author

fjpolo commented Jun 25, 2024

It's finally working as expected. PR contains files from Cheats PR because I rebased form there

@fjpolo
Copy link
Contributor Author

fjpolo commented Jul 8, 2024

Will be added in #88

@fjpolo fjpolo closed this Jul 8, 2024
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.

3 participants