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/rmet 2912/ use coroutines for bitmap work #11

Closed
wants to merge 2 commits into from

Conversation

alexgerardojacinto
Copy link
Collaborator

Description

Context

Type of changes

  • Fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Refactor (cosmetic changes)
  • Breaking change (change that would cause existing functionality to not work as expected)

Platforms affected

  • Android
  • iOS
  • JavaScript

Tests

Screenshots (if appropriate)

Checklist

  • Pull request title follows the format RNMT-XXXX <title>
  • Code follows code style of this project
  • CHANGELOG.md file is correctly updated
  • Changes require an update to the documentation
    • Documentation has been updated accordingly

Context: In Android, when working with bitmaps, which is a type of image processing, it is recommended to use coroutines so that the main UI thread isn't blocked.

Dispatchers.Default is used because it is optimized for tasks that are CPU intensive, like image processing and transformation, since it provides a limited number of threads that is equal to the number of CPU cores.

More info: https://developer.android.com/develop/ui/views/graphics and https://developer.android.com/kotlin/coroutines/coroutines-adv
@alexgerardojacinto alexgerardojacinto changed the base branch from main to development December 13, 2023 23:12
Copy link

@alexgerardojacinto alexgerardojacinto changed the title Feat/rmet 2912/screen tablet Feat/rmet 2912/ use coroutines for bitmap work Dec 19, 2023
@alexgerardojacinto alexgerardojacinto deleted the feat/RMET-2912/screen-tablet branch January 10, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant