NoiseView icon indicating copy to clipboard operation
NoiseView copied to clipboard

Screen goes black after few seconds

Open rostopira opened this issue 8 years ago • 5 comments

I've tried both Google Play version and compiled from sources. It works for few seconds, then screen goes black and process dies. No errors or crashes in logcat. Idunno wtf is that, but you probably need to know about it ¯_(ツ)_/¯ Device: Galaxy Note 4 with stock Marshmallow UPD: Same thing on Xperia Z1 with stock Lollipop

rostopira avatar Oct 27 '17 00:10 rostopira

I've fully rewrote regular noise and left scratches untouched. It works now and looks better IMHO, but needs few tweaks before it could be shown to people without shame :D

rostopira avatar Oct 27 '17 01:10 rostopira

Thank you! Will you make pull request after you're done?

hypeapps avatar Oct 27 '17 10:10 hypeapps

@hypeapps yep, but I need few hours to finish But here really a lot of changes. Noise are created with RenderScript now. Also, I've refactored NoiseRenderable a bit.

rostopira avatar Oct 27 '17 10:10 rostopira

OK, so it's early for PR, because of few things. Scratches became invisible, because intensity slider in my code controls dots frequency, and in your code it controls transparency. Also, I've considered that scale 1f renders frame in original resolution, and 0.5f renders quarter of it and scales up. Scale down - noise dots bigger. I've done everything wrong :D Checkout fork to see, what I'm talking about: https://github.com/rostopira/NoiseView My suggestion is to make separate options for regular noise and scratches

rostopira avatar Oct 27 '17 10:10 rostopira

I really like your changes. What do you mean by make seperate options? Will you edit all fields of view? Can you explain more?

hypeapps avatar Nov 03 '17 22:11 hypeapps