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

About trap 6 on macOS #4

Open
misha-plus opened this issue Feb 13, 2019 · 4 comments
Open

About trap 6 on macOS #4

misha-plus opened this issue Feb 13, 2019 · 4 comments

Comments

@misha-plus
Copy link

Hello.
I run v0.3.0 -- it is good.
But on v0.4 I have following problem

Reading 235520.jpg...Ok
Reading scale2.0x_model.json...Ok
[scale2] %.1f%% - ELA: 0.0 - ETA: 00:00:01 - MEM: 596:31:23 Abort trap: 6

I fastly looked to the code it uses half precision but on my MacBook Pro Mid 2013 with HD Graphics 4000 there is no half precision -- can this be the cause of the error?

@misha-plus
Copy link
Author

I've changed half to float -- it helps

@soywiz
Copy link
Member

soywiz commented Feb 14, 2019

Can you make a PR?

@misha-plus
Copy link
Author

I can make it but this can make kernel slower on NVidia or AMD. Is this ok?

@soywiz
Copy link
Member

soywiz commented Feb 14, 2019

Yeah, it is ok.
If you are using a mac, I would suggest you to try https://github.com/imxieyi/waifu2x-mac it is an order of magnitude faster. I was not able to optimize it further using OpenCL and didn't investigate more.
So I guess it is better to have something that works on most of the systems out of the box including linux, and if half-floats are not available everywhere better to use floats for now. And detect if half floats are available just as an optimization.

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

No branches or pull requests

2 participants