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

Added paint brush tools #4543

Merged
merged 134 commits into from
Nov 2, 2022
Merged

Added paint brush tools #4543

merged 134 commits into from
Nov 2, 2022

Conversation

bsekachev
Copy link
Member

@bsekachev bsekachev commented Apr 4, 2022

Motivation and context

Resolved #1849
Resolved #4868

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@kkmehta03
Copy link

i tried checking out this branch but couldn't find the brush tool for some reason.
i did

docker-compose rm
docker-compose up

am i missing something here?

@bsekachev
Copy link
Member Author

/check

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2022

❌ Some checks failed
📄 See logs here

@sizov-kirill
Copy link
Contributor

sizov-kirill commented Oct 24, 2022

@bsekachev Do you see any reason why a failed test runs infinitely or it deserves separate issue ticket?

@bsekachev
Copy link
Member Author

/check

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2022

❌ Some checks failed
📄 See logs here

@bsekachev
Copy link
Member Author

/check

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2022

✔️ All checks completed successfully
📄 See logs here

@bsekachev
Copy link
Member Author

@bsekachev Do you see any reason why a failed test runs infinitely or it deserves separate issue ticket?

@kirill-sizov I am not sure why they are infinitely, but now they are ok

@bsekachev bsekachev mentioned this pull request Oct 31, 2022
8 tasks
@bsekachev
Copy link
Member Author

/check

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2022

✔️ All checks completed successfully
📄 See logs here

@nmanovic
Copy link
Contributor

nmanovic commented Nov 2, 2022

/check

@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2022

✔️ All checks completed successfully
📄 See logs here

@nmanovic nmanovic merged commit f62aa27 into develop Nov 2, 2022
@nmanovic nmanovic deleted the bs/masks branch November 2, 2022 05:38
nmanovic added a commit that referenced this pull request Nov 30, 2022
nmanovic added a commit that referenced this pull request Nov 30, 2022
PR #4543 didn't add a new migration.
The migration wasn't critical. At least it was not led to any problems.
@nmanovic nmanovic mentioned this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
10 participants