-
Couldn't load subscription status.
- Fork 10.6k
[benchmark] Support multiple regular expression filters in Benchmark_Driver #9026
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
Conversation
|
@palimondo Can this new option be followed by other options or other arguments? If not, may be it is better to use multiple occurrences of Also, have you thought about using substrings or even regexes instead of string-based prefixes? |
|
The relationship between the dashed argument '-f' and the positional arguments is unclear. Is the prefix string an argument of the '-f' option or does '-f' change the meaning of all positional arguments. |
|
I was hoping it would work like that, requiring multiple gave out… I have to look some more at The |
|
Ah, I got it. I shouldn’t have used |
|
@swiftix Re: substrings and regexes, I thought about that. Got scared and ran away… I mean, prefix works for my needs, but we could play with it, now that it’s in. Do you have specific use case in mind, or just thinking in the abstract? |
Updated usage documentation to reflect this.
|
OK, I think this should work. Old format doesn’t work anymore and verifies that the I promote this feature in initial help (given that the subcommands arguments are hidden in their respective subcommand help...): And now it works like this: |
|
Please review? |
|
@swift-ci please python lint |
|
Let's run the benchmarks just to be sure, then I will merge it. @swift-ci Please benchmark |
|
He rebased this, so once more with feeling: // @shahmishal @swift-ci please benchmark |
Build comment file:Optimized (O) No Changes (269)
Regression (1)
No Changes (268)
|
|
@swift-ci Please smoke test |
|
Smoke test failed on python lint. But the "@swift-ci python lint" was successful? 🤷♂️🤦♂️ I have amended the python lint fixes and force pushed. @shahmishal Please rerun the smoke test. |
|
@swift-ci please smoke test |
|
Can this be merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@palimondo thanks!! |
This is a minor follow-up to #8975 Fix SR-4598 Add option to run subset of benchmarks matching a prefix.
It adds option to specify multiple prefixes and run all tests matching them. For example:
runs all the benchmarks for the
Sequenceinterface.