Skip to content

Feat: Adding UI Event Recorder - #1259

Merged
maharshi-gor merged 4 commits into
fury-gl:masterfrom
ganimtron-10:eventcounter
Jul 3, 2026
Merged

Feat: Adding UI Event Recorder#1259
maharshi-gor merged 4 commits into
fury-gl:masterfrom
ganimtron-10:eventcounter

Conversation

@ganimtron-10

@ganimtron-10 ganimtron-10 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Adding UI Event Recorder to record, replay and simulate UI interaction tests

Fixes #1122

@ganimtron-10
ganimtron-10 force-pushed the eventcounter branch 5 times, most recently from 710b019 to 9fba9ed Compare June 27, 2026 12:10
@ganimtron-10

Copy link
Copy Markdown
Contributor Author

This works fine in Show Simulation Mode with glfw window but it throws the error as in the CI for offscreen and it behaves weirdly. I initially thought its due to Panel2D but ListBox2D works and only these four cause the issue.

image

I am still figuring out why specifically, @maharshi-gor do you have any idea what might cause this?
Also if you want to make release with this change, let me know will remove these specific testcase from this PR and create a new PR with these testcase and we can keep figuring that part out.

@maharshi-gor

Copy link
Copy Markdown
Contributor

I have no idea. But let me try locally and I will get back to you @ganimtron-10

@maharshi-gor

Copy link
Copy Markdown
Contributor

Hello @ganimtron-10

I have an idea why it is causing and I fixed it on my local I can push the commit if you want.

There was a small bug that we were dispatching the event before the step_count getting updated in testing.py.

Now, we had everything correct just few frames back. So simulation works nicely but the validation where exact pixel matters became problematic.

Let me know if that make sense.

@ganimtron-10

Copy link
Copy Markdown
Contributor Author

Sure, Please push the change.

It didn't completely make sense the last part but might make after seeing your change.

@maharshi-gor

Copy link
Copy Markdown
Contributor

Hey @ganimtron-10 I have pushed the PR please check

@ganimtron-10
ganimtron-10 force-pushed the eventcounter branch 2 times, most recently from aa0a1ff to 7370abf Compare July 2, 2026 02:07
@ganimtron-10
ganimtron-10 marked this pull request as ready for review July 2, 2026 02:37
@maharshi-gor

Copy link
Copy Markdown
Contributor

Hey @ganimtron-10 these are taking too long and operations are cancelled can we do something about them

@ganimtron-10

Copy link
Copy Markdown
Contributor Author

Hi @maharshi-gor, that what I am looking into, Ideally it shouldn't take this long.

@maharshi-gor
maharshi-gor merged commit 1cab514 into fury-gl:master Jul 3, 2026
22 checks passed
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.

Re-implement UI Event Recorder and Counter

2 participants