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

Segment display emulator #417

Merged
merged 25 commits into from
Apr 18, 2021
Merged

Segment display emulator #417

merged 25 commits into from
Apr 18, 2021

Conversation

qcapen
Copy link
Collaborator

@qcapen qcapen commented Apr 15, 2021

PR for a new virtual segment display emulator (also related PRs for MPF-MC and docs). Uses virtual hardware platform segment display to update new segment_display_emulator widget in the MC. Supports the new color feature added for the Fast segment displays. The segment display widgets can be updated using the segment_display_player.

The widget_player can be used to update the new segment_display_emulator widgets, but it is not the recommended way to control these as the widget_player destroys and re-creates widgets when they are updated and that is a much more expensive operation than using the segment_display_player.

Still have some more advanced features to add (transition effects, individual character colors), but wanted to get this pulled into the code base first.

@qcapen qcapen self-assigned this Apr 15, 2021
@qcapen
Copy link
Collaborator Author

qcapen commented Apr 15, 2021

It looks like my "fix" to the glow effects to support Windows breaks it on linux. I should just revert that for now and leave that as a separate issue as it is not directly related to this PR.

Copy link
Member

@seanirby seanirby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@qcapen qcapen merged commit 5b62928 into dev Apr 18, 2021
@qcapen qcapen deleted the segment_display_emulator branch April 18, 2021 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants