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: add callback to apply kornia augmentations #862

Merged
merged 12 commits into from Jul 15, 2020
Merged

feat: add callback to apply kornia augmentations #862

merged 12 commits into from Jul 15, 2020

Conversation

bagxi
Copy link
Member

@bagxi bagxi commented Jun 28, 2020

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you read the contribution guide?
  • Did you check the code style? catalyst-make-codestyle && catalyst-check-codestyle (pip install -U catalyst-codestyle).
  • Did you make sure to update the docs? We use Google format for all the methods and classes.
  • Did you check the docs with make check-docs?
  • Did you write any new necessary tests?
  • Did you add your new functionality to the docs?
  • Did you update the CHANGELOG?
  • You can use 'Login as guest' to see Teamcity build logs.

Description

Related Issue

Type of Change

  • Examples / docs / tutorials / contributors update
  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves an existing feature)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

@bagxi bagxi requested a review from Scitator June 28, 2020 15:19
@bagxi bagxi force-pushed the kornia branch 2 times, most recently from c1819c1 to b443c97 Compare June 28, 2020 15:24
@bagxi bagxi added the WIP Pull request is under construction label Jun 28, 2020
@mergify
Copy link

mergify bot commented Jun 28, 2020

This pull request is now in conflicts. @bagxi, could you fix it? 🙏

@bagxi bagxi linked an issue Jun 28, 2020 that may be closed by this pull request
@bagxi bagxi self-assigned this Jun 29, 2020
@mergify
Copy link

mergify bot commented Jun 29, 2020

This pull request is now in conflicts. @bagxi, could you fix it? 🙏

@mergify
Copy link

mergify bot commented Jul 5, 2020

This pull request is now in conflicts. @bagxi, could you fix it? 🙏

@bagxi bagxi force-pushed the kornia branch 2 times, most recently from 9fc7a6a to 95a8db4 Compare July 12, 2020 16:25
tests/_tests_scripts/cv_z_kornia.py Show resolved Hide resolved
tests/_tests_scripts/cv_z_kornia.py Show resolved Hide resolved
tests/_tests_scripts/cv_z_kornia.py Show resolved Hide resolved
tests/_tests_scripts/cv_z_kornia.py Outdated Show resolved Hide resolved
@bagxi bagxi force-pushed the kornia branch 8 times, most recently from 035bdfc to d356ad7 Compare July 13, 2020 20:25
@bagxi bagxi force-pushed the kornia branch 13 times, most recently from d1945c6 to cf985f8 Compare July 14, 2020 19:11
catalyst/contrib/dl/callbacks/__init__.py Outdated Show resolved Hide resolved
catalyst/contrib/registry.py Outdated Show resolved Hide resolved
@Scitator Scitator merged commit e0be118 into master Jul 15, 2020
@mergify mergify bot deleted the kornia branch July 15, 2020 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Pull request is under construction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kornia integration
2 participants