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

Haptic player error #1790

Open
Sjmarf opened this issue Feb 22, 2025 · 5 comments
Open

Haptic player error #1790

Sjmarf opened this issue Feb 22, 2025 · 5 comments
Assignees
Labels
pending repro This issue is on hold pending consistent reproduction.

Comments

@Sjmarf
Copy link
Member

Sjmarf commented Feb 22, 2025

Got this when switching back to Mlem from another app

Optional(Error Domain=com.apple.CoreHaptics Code=-4810 "(null)" UserInfo={Error =_player != nil}) (Mlem/HapticManager.swift, startEngine():51)
@Sjmarf Sjmarf added this to the v2.0 milestone Feb 22, 2025
@EricBAndrews EricBAndrews self-assigned this Feb 22, 2025
@EricBAndrews
Copy link
Member

Did haptics continue to work after this error?

@Sjmarf
Copy link
Member Author

Sjmarf commented Feb 22, 2025

Didn't check, sorry :/

@EricBAndrews EricBAndrews added the pending repro This issue is on hold pending consistent reproduction. label Feb 22, 2025
@EricBAndrews
Copy link
Member

EricBAndrews commented Feb 22, 2025

I'm having some trouble getting this to happen--I think it's probably one of those things where you have to catch the engine in just the wrong state. If the haptics continue to work, we should just silence the error so it doesn't pop a toast; otherwise the engine needs some more sophisticated failure recovery

@EricBAndrews EricBAndrews removed their assignment Feb 22, 2025
@EricBAndrews
Copy link
Member

I got a repro! Haptics stop working after this error occurs.

@EricBAndrews
Copy link
Member

Moving this off 2.0 because #1829 should at least band-aid fix it, and even if it doesn't the error is super rare and doesn't break core functionality (if a user needs those haptics, they can always just restart the app).

Keeping this issue open until either we figure out the root cause or it doesn't show up for long enough that we can call it squashed.

@EricBAndrews EricBAndrews removed this from the v2.0 milestone Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending repro This issue is on hold pending consistent reproduction.
Projects
None yet
Development

No branches or pull requests

2 participants