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(CanvasFallback): Created 2d CanvasFallback #12

Merged
merged 3 commits into from
Mar 7, 2024

Conversation

gaiborjosue
Copy link
Collaborator

Hello @haehn,

Major changes include:
This PR implements the 2d Canvas Fallback we discussed during this week's meeting. It also removes duplicated code from currently available frameworks.

Minor changes include:

  • I added a Boostlet hint for SAM in OSD. I saw it was intended in the past, so I uncommented the alert.
  • I added the test images taken from local testing sessions to the .gitignore so that developers don't have to delete the testing images each time they want to collaborate with the main branch.

Overall, this was my impression of the fallback, but please feel free to suggest improvements or guide the development; thanks!

@gaiborjosue gaiborjosue added the enhancement New feature or request label Mar 1, 2024
@gaiborjosue gaiborjosue requested a review from haehn March 1, 2024 18:52
@gaiborjosue gaiborjosue self-assigned this Mar 1, 2024
Copy link

github-actions bot commented Mar 1, 2024

✅ Test Passed ✅

Hi there @gaiborjosue! It looks like the tests passed successfully. 😃

Please check the https://github.com/mpsych/boostlet/actions/runs/8115950652 for more details.

If you want to access the screenshots taken for testing, you can do it through the developer's Claudinary management account, or through the workflow's artifacts.

@gaiborjosue gaiborjosue changed the title feat(CanvasFallback): Created 2d Canvas Fallback feat(CanvasFallback): Created 2d CanvasFallback Mar 1, 2024
Copy link
Contributor

@haehn haehn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super - look at all the removed code duplication!! Well done @gaiborjosue !

@haehn haehn merged commit 33848e4 into mpsych:main Mar 7, 2024
5 of 6 checks passed
haehn pushed a commit that referenced this pull request Apr 23, 2024
…odules

Canvas fallback boxcraft submodules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants