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

Use FastBlur as a fallback upon RenderScript failure #21

Merged
merged 1 commit into from
Feb 27, 2016
Merged

Use FastBlur as a fallback upon RenderScript failure #21

merged 1 commit into from
Feb 27, 2016

Conversation

Pkmmte
Copy link
Contributor

@Pkmmte Pkmmte commented Feb 25, 2016

Certain low-end devices crash when calling RenderScript.create(context).

A solution to this issue is beyond my control so, instead, I created a
fallback solution. It now uses the FastBlur class when RenderScript
fails.
http://trickyandroid.com/advanced-blurring-techniques/

Cons:

  • Memory usage is higher than RenderScript on affected devices.

Pros:

  • It no longer crashes on affected devices.
  • The process remains the exact same on devices which it previously
    worked fine on.

Certain low-end devices crash when calling RenderScript.create(context).

A solution to this issue is beyond my control so, instead, I created a
fallback solution. It now uses the FastBlur class when RenderScript
fails.
http://trickyandroid.com/advanced-blurring-techniques/

Cons:
- Memory usage is higher than RenderScript on affected devices.

Pros:
- It no longer crashes on affected devices.
- The process remains the exact same on devices which it previously
worked fine on.
@Pkmmte
Copy link
Contributor Author

Pkmmte commented Feb 25, 2016

This should fix issue #16

@wasabeef
Copy link
Owner

@Pkmmte

Thank you.

I will check.

@Pkmmte
Copy link
Contributor Author

Pkmmte commented Feb 25, 2016

You can also use FastBlur to provide a "lite" version of this Blurry library without the renderscriptSupportModeEnabled requirement. I'm using a custom version of Blurry in my project that uses the FastBlur class on APIs below 17 (and non-support RS in 17+) and it has made my .apk size 2.5MB smaller!

I can include that change too, if you'd like. :D

wasabeef added a commit that referenced this pull request Feb 27, 2016
Use FastBlur as a fallback upon RenderScript failure
@wasabeef wasabeef merged commit 8a2f341 into wasabeef:master Feb 27, 2016
@wasabeef wasabeef added this to the 1.1.0 milestone Feb 27, 2016
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.

2 participants