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

feat: provide option to start with recording page #660

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GitMensch
Copy link
Contributor

This is a draft to showcase the working feature.

The main question is: Do we want a single option (as in this PR) that opens the recording page when no files are given (then we may want to output an error if the QStringList files is not empty); or do we want an option like "openPage" with a set of names that works in general (also for files opened, which then needs to be postponed after a possible load is done to switch from the summary page to the specified one).

... and of course I'm open to a better name than record or openPage.

after everything is clear the README will get updated as well to include the new option in the help output

@milianw
Copy link
Member

milianw commented Jul 5, 2024

Imo, --record sounds too similar to perf record, meaning it would expect it to directly start recording whatever else is passed after it. But this would require (much) more work, with little gain - one can just copy'n'paste the record step into a script and only visualize the data afterwards after all. That said, I'm not opposed to someone working on that if there is a desire for such functionality.

For your use-case, I think we should rather approach it differently, i.e. like you say here:

an option like "openPage" with a set of names that works in general

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