Skip to content

Add bxt_cap_time_*#68

Draft
khanghugo wants to merge 2 commits intoYaLTeR:masterfrom
khanghugo:cap-time
Draft

Add bxt_cap_time_*#68
khanghugo wants to merge 2 commits intoYaLTeR:masterfrom
khanghugo:cap-time

Conversation

@khanghugo
Copy link
Copy Markdown
Contributor

@khanghugo khanghugo commented Jun 12, 2023

I was working with campath and sometimes I just want to capture a segment. Currently there is no way to scrub through demo quickly and this just seems to do the trick. Tested with normal and sampling capture.

The idea is just to have things working around playdemo and very easy to automate with.

@khanghugo
Copy link
Copy Markdown
Contributor Author

khanghugo commented Jun 14, 2023

I am not sure how the r_norefresh 1 still makes skipping demo not fast enough. Right now, it is about twice as fast as timedemo when I think it is supposed to be faster. I guess this can be dependent on _bxt_norefresh or maybe reimplementation here for bxt-rs.

@khanghugo khanghugo marked this pull request as draft June 15, 2023 12:13
@khanghugo
Copy link
Copy Markdown
Contributor Author

This can be made as a different module I think. Or maybe it can stay vaporware.

Copy link
Copy Markdown
Owner

@YaLTeR YaLTeR left a comment

Choose a reason for hiding this comment

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

Yea idk this does make sense, need to figure out how to best do it.

}

capture_video_per_demo::stop(marker);
TIME.set(marker, 0.);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Needs to be zeroed on cap start, not stop.

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.

2 participants