imxieyi / waifu2x-ios

iOS Core ML implementation of waifu2x
MIT License
551 stars 58 forks source link

[Feature Request]macOS版本的脚本或Automator支持/Scripting or Automator support in macOS version #17

Closed i0ntempest closed 4 years ago

i0ntempest commented 4 years ago

首先感谢作者,很高兴macOS上终于有一个能替代windows平台上各种waifu2x的app了。如果能用脚本或者automator来批量处理图片放大到不同大小就完美了,另外有3倍放大的话就更完美了。

imxieyi commented 4 years ago

会考虑加入命令行支持。加入3倍意义不大,waifu2x-caffe的任意倍数实现方式是放大后再缩小(见nagadomi/waifu2x#275),缩小部分可以使用其他软件实现。

i0ntempest commented 4 years ago

了解了,我写脚本的时候调用imagemagick好了

imxieyi commented 4 years ago

通过App Store发布的App会被沙盒严格限制权限,即使是通过命令行运行也无法幸免。所以就算加入命令行也只能支持在预设的文件夹(下载、图片、音乐、视频)下读写数据。automator的实现较为复杂而且相关文档极少。暂时只能支持命令行读写下载目录。考虑到会使用命令行的一般都是power user,这样的限制应该没有太大问题。

i0ntempest commented 4 years ago

这个的话,开启full disk access应该可破?AppStore里面的文件管理器都会问这个权限的

imxieyi commented 4 years ago

开了full disk access并没有用,可能还是得手动授权单独的文件夹。

i0ntempest commented 4 years ago

这样吗,手动授权文件夹有用的话也不是不行啊 另外可以的话建议加一个安装(symlink)到/usr/local/bin/的按钮

imxieyi commented 4 years ago

因为rpath的问题symlink不可行。只能通过完整路径运行或者将app安装目录加入PATH。

i0ntempest commented 4 years ago

意思是通过相对路径link了别的库吗,这样的话应该可以摸个script当launcher然后把它扔进bin/就可以了,我摸一个PR给你也成

i0ntempest commented 4 years ago

感谢更新,刚刚试了下新的cli功能,然后摸了一个简单的脚本

#!/bin/sh

#Edit this line to define a path to waifu2x.app
WAIFU2X_APP="/Applications/waifu2x.app"

if [ ! -d "${WAIFU2X_APP}" ] ; then
    echo "waifu2x.app not found";exit 1;
fi

"${WAIFU2X_APP}/Contents/MacOS/waifu2x" "$@"

您想的话可以直接内置进app提供一个安装到/usr/local/bin/waifu2x的功能 没别的事的话可以close了

imxieyi commented 4 years ago

App Store中的应用是不允许获取更高权限的(apple自己的应用除外),所以自动安装可能性不大。不过感谢你的脚本,我可以把它放进wiki页面中。

i0ntempest commented 4 years ago

好,不过作为参考的话store里的BBEdit有一个安装cli tools的功能应该可以学过来

imxieyi commented 4 years ago

似乎可以向apple申请exception,然而众所周知和apple打交道相当费劲,目前的方式应该不会给会用命令行的人带来任何障碍,所以不太值得去折腾。