-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 PhotometricDistortion #485
Conversation
Codecov Report
@@ Coverage Diff @@
## master #485 +/- ##
==========================================
+ Coverage 80.99% 81.05% +0.05%
==========================================
Files 138 138
Lines 9425 9474 +49
Branches 1510 1518 +8
==========================================
+ Hits 7634 7679 +45
- Misses 1448 1451 +3
- Partials 343 344 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
||
|
||
@PIPELINES.register_module() | ||
class PhotoMetricDistortion(object): |
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.
PhotoMetric
-> Photometric
It is derived from word photometry
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.
btw (object)
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.
Actually, PhotoMetricDistortion is copied from mmdet. Shall we keep the name the same?
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.
no
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.
We can raise an issue there~
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.
cc. @yhcao6
Nevermind, let's consider optimization later. |
5. random hue | ||
6. convert color from HSV to BGR | ||
7. random contrast (mode 1) | ||
8. randomly swap channels |
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.
Step 8 is missing
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.
cc. @yhcao6
* add PhotoMetricDistortion * update saturation_hue * remove (object) * mv dice to call * add random swap channels
* add PhotoMetricDistortion * update saturation_hue * remove (object) * mv dice to call * add random swap channels
No description provided.