Skip to content

sinclair/spectrum_v.cpp: Added ULA snow effect support [holub, spectramine] #13963

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

holub
Copy link
Contributor

@holub holub commented Jul 17, 2025

There is an issue with precise timings. Will try to sort that out next time.

@holub holub marked this pull request as draft July 18, 2025 12:32
@holub holub changed the title sinclair/spectrum_v.cpp: Added preliminary ULA snow effect support sinclair/spectrum_v.cpp: Added preliminary ULA snow effect support for Spectrum48 Jul 19, 2025
@holub holub marked this pull request as ready for review July 19, 2025 20:57
@holub
Copy link
Contributor Author

holub commented Jul 19, 2025

I'm done with 48 machine. 128/+2 works somehow but not precise...

@holub holub changed the title sinclair/spectrum_v.cpp: Added preliminary ULA snow effect support for Spectrum48 sinclair/spectrum_v.cpp: Added preliminary ULA snow effect support Jul 19, 2025
@holub
Copy link
Contributor Author

holub commented Jul 19, 2025

Let it snow!

@holub holub changed the title sinclair/spectrum_v.cpp: Added preliminary ULA snow effect support sinclair/spectrum_v.cpp: Added ULA snow effect support Jul 19, 2025
@holub holub changed the title sinclair/spectrum_v.cpp: Added ULA snow effect support sinclair/spectrum_v.cpp: Added ULA snow effect support [holub, spectramine] Jul 20, 2025
@ICEknigh7
Copy link
Contributor

Can confirm that the snow effect seen after Strider crashes looks identical as in real hardware:
image
image

The screen glitches in The Ninja Warriors look a bit different but those may depend on something else than the known snow effect:
image
image
image
image
image
image
image

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