-
Notifications
You must be signed in to change notification settings - Fork 133
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
Add detection support for 4 more formats #550
Add detection support for 4 more formats #550
Conversation
* lfw * open_images * vgg_face2 * widerface
|
||
def fail(self, requirement: str) -> NoReturn: | ||
""" | ||
Places a requirement that is never met. `requirement` must contain | ||
a human-readable description of the requirement. | ||
""" | ||
self._start_requirement() | ||
self._start_requirement("fail") |
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.
"a requirement ('fail') can't be placed directly within an 'alternatives' block"
Still sound strange, IMO. Not a blocker, anyway.
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.
🤷♂️ It's a message for developers. I think they can figure it out.
The new name more directly represents what the function does, and is less visually similar to `alternatives`.
This kind of value is called `requirement_desc` elsewhere.
Summary
The formats in question are
lfw
,open_images
,vgg_face2
andwiderface
. As supporting infrastructure, add alternative requirement support.Also, add low-level format detection tests, which I should really have done earlier.
How to test
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.