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

Sliderules: A dynamic ISA reference table #329

Draft
wants to merge 64 commits into
base: master
Choose a base branch
from

Conversation

raccog
Copy link
Contributor

@raccog raccog commented Dec 21, 2023

Draft branch for #297. Progress on implementing this feature will be posted here. Discussion on plans for this feature will be posted in issue #297.

Currently, I have partially implemented the encoding table (without pseudo-instructions) and am working on its filtering system. I have not started implementing the decoding table, yet.

Here is an example of the current encoding table and its filtering system: (note that the checkboxes are not used because they are not working yet)

Sliderules.-.2024-01-06.21-04-16.mp4

Since release-based actions currently require @mortbopet's account
details, these actions automatically fail when run on forks. These have
been prevented in this commit by adding a conditional that runs them
only for @mortbopet's repo.

This could be expanded in the future so that forks can use their own
account info to create releases.
The sliderules tab will now use two "models" that contains encoding
and decoding data. Custom Qt widgets and the Qt signal/slot mechanism
will be used for all data/filter updates.

For example, a signal that is emitted when the encoding table's ISA
family is changed is connected only to a slot in the encoding model. The
model would then emit its own signal that is connected to any UI slot
that needs to display the changed data. This flow of `User Interaction ->
Model Update -> UI Update` will ensure there are no recursive loops in the
signal/slot connection tree.
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.

1 participant