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

Passed blarrg's cgb_sound tests, and other sound things 🔊 #286

Merged
merged 34 commits into from
Apr 27, 2019

Conversation

torch2424
Copy link
Owner

@torch2424 torch2424 commented Apr 24, 2019

opens #197
opens #283

This was super hard. I was doing a lot of travelling in between, so I didn't have a lot of time to work on this. But for real, it was taking me like 6 hours per subtest. Sometimes, even subtests of the subtest. 😢 But I'm so glad these tests are over 😄

  • Makes the fixes to pass the tests.
  • Slightly fixes crackling mentioned in WasmBoy 1.0 GB/GBC Roadmap #197
  • Sets up the tests for the cgb_sound going forward 😄

Example in the README

cgb_sound golden

@torch2424 torch2424 added the Core/Wasm Issues concerning the core library of Wasmboy label Apr 24, 2019
@torch2424 torch2424 self-assigned this Apr 24, 2019
@torch2424 torch2424 merged commit 06dee7a into master Apr 27, 2019
@torch2424 torch2424 deleted the sound-tests branch April 27, 2019 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core/Wasm Issues concerning the core library of Wasmboy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant