DeadSix27 / waifu2x-converter-cpp

Improved fork of Waifu2X C++ using OpenCL and OpenCV
MIT License
792 stars 86 forks source link

github: add basic workflow implementation #208

Closed DeadSix27 closed 4 years ago

DeadSix27 commented 5 years ago

This runner simply builds the program on Linux and a Windows and additionally uploads the windows build artifacts

This should be merged after: #207 to fix the windows build from failing.

EDIT: We should consider possibly limiting the triggers, but generally I think on every PR and every commit and every release is fine? we don't have that many PR's nor commits, so I imagine its not a big deal.

EDIT2: Also thought about disabling CUDA (on windows), it takes like 12min to install on that VM. But then again, same thing applies, we really don't have that frequent changes, so does it matter?

EDIT3: Removed cuda, since that codebase is rather stale and the aforementioned increase it check speed is totally worth it.