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

Added Frame Range Option to Convert Tool #1295

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

laurentiu-erhan-arm
Copy link
Contributor

Frame range option allows to only retrieve relevant data and reduce waiting time, especially useful for bigger traces

Change-Id: I3b7298a20c41d2c67a59069ddf31961f37a3b12d

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 56820.

@@ -72,6 +72,9 @@ static void PrintUsage(const char* exe_name)
GFXRECON_WRITE_CONSOLE(
" --file-per-frame\tCreates a new file for every frame processed. Frame number is added as a suffix");
GFXRECON_WRITE_CONSOLE(" \tto the output file name.");
GFXRECON_WRITE_CONSOLE(
" --frame-range\t\tFrame range to be converted. In order to retrieve Trim Trace State, frame 0 has "
"to be in frame range.");
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be good to tell the user the format for specifying the frame range here.

Copy link
Contributor Author

@laurentiu-erhan-arm laurentiu-erhan-arm Oct 5, 2023

Choose a reason for hiding this comment

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

I have updated the help message to:

--frame-range <N1[-N2][,...]>
                        Frame ranges to be converted. In order to retrieve trim trace state, frame 0 has to be in frame range.
                        Frame ranges should be specified in ascending order and cannot overlap. Frame numbering is zero-indexed and inclusive.
                        Example: 0-2,5,8-10 will generate data for 7 frames.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 56850.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3347 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 69624.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3415 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3415 failed.

@bradgrantham-lunarg bradgrantham-lunarg added the P2 A high-priority code maintenance issue or a functional problem that is recoverable or not a crash. label Oct 30, 2023
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 100998.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3625 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3625 passed.

Frame range option allows to only retrieve relevant data and reduce waiting time, especially useful for  bigger traces

Change-Id: I3b7298a20c41d2c67a59069ddf31961f37a3b12d
@laurentiu-erhan-arm laurentiu-erhan-arm force-pushed the convertFrameRangeSupport branch from fc1baa4 to ebaf99e Compare May 8, 2024 11:46
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 181553.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4125 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4125 failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 A high-priority code maintenance issue or a functional problem that is recoverable or not a crash.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants