Skip to content

Add minimal Recording #112

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 13 commits into
base: master
Choose a base branch
from
Open

Conversation

53845714nF
Copy link
Contributor

I know it is a bit bigger PR.

Recording:

I create a minimal Recording Feature, it creates a Python Script with the Click you make.
You can start the recording with Crtl + R.
With Alt and Mouse Click, it writes existsAndVisible to the Python Script.
The Recording Stops with clicking Crtl + R again.

It is not perfect, but it caves a small Sample.
Which can then be used for future work.

Path:

It also creates new features in the Path.

Properties:

I have added a feature where you can filter on Properties with the following Syntax (property=value) for example: (source=qrc:/menu/back.png)

Number/Index:

There is also a new feature in the Path where you can select the x element with a # for example: mainWindow/"Example Text"#2.

@53845714nF 53845714nF force-pushed the add_record_clicks branch 4 times, most recently from 9b687de to f25b014 Compare May 6, 2024 12:30
@53845714nF 53845714nF force-pushed the add_record_clicks branch 2 times, most recently from d583676 to d170924 Compare May 6, 2024 12:38
@53845714nF 53845714nF force-pushed the add_record_clicks branch from 8f3ed19 to cab36d0 Compare May 6, 2024 13:07
@faaxm
Copy link
Owner

faaxm commented Apr 9, 2025

The recording feature is really exciting and I appreciate the effort put into this implementation. I am currently exploring some changes to the Spix architecture, and at the moment the tight integration with QtScene limits flexibility. I see a design where users can easily swap out different implementations for recording and tracking instead of having a fixed Python output.

For now, I would like to keep this PR on record for reference and revisit some of these ideas once the architecture for a more generic event tracking feature is in place. Thank you for your work on this and I look forward to further evolving the design.

@faaxm faaxm added the on-hold Waiting for other PRs to be merged first label Apr 10, 2025
@53845714nF
Copy link
Contributor Author

OK, thank you. The project is now free time for me. So I don't know how much time I can put into changing the architecture. Although I do find the idea of making the Framework project more agnostic exciting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on-hold Waiting for other PRs to be merged first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants