近期优化 follow up PR #77 #78 #74

web3nomad commented 3 months ago

基于最近 merge 以后的进一步优化 #77 #78 #74

[ ] crates/cron 名字似乎和 rust 官方的 cron 重名了，这个不大好
[ ] DataLocation 这个数据库表需要测试一下客户端升级的时候有没有问题，以及 review 下是否需要进一步优化
[ ] settings.tsx 设置页面确认，配置 aws s3 keys 是否足够
~crates/storage-macro/src/lib.rs 里面很多方法的参数是不是可以改成 AsRef<Path>~
~suggestions 接口需要修改，现在逐个遍历和读取文件太慢了~ (这个功能需要重写)
[ ] protocol 其实应该改成输入 hash 而不是输入完整的 artifacts 或者 files 目录
[ ] crates/storage/src/traits.rs 里面 read_to_string 等需要加一些注释，解释下方法的作用
[ ] https://github.com/bmrlab/gendam/pull/77/files#r1656822855 这里可以考虑用临时目录来代替先 000-tmp 再删除的流程。save_video_audio 和 save_video_frames 都可以这么优化。
[ ] 全局变量维护的值（比如 CURRENT_LIBRARY），和 CTX 里面维护的值，要合并或重新设计下。

ZingerLittleBee commented 3 months ago

crates/storage-macro/src/lib.rs 里面很多方法的参数是不是可以改成 AsRef

一开始入参就是 impl AsRef<Path>，但是有生命周期相关的问题（记不清了），没解决掉，就改成 PathBuf 了

RustLoverTheCoder commented 3 months ago

crates/cron 名字似乎和 rust 官方的 cron 重名了，这个不大好

新的名称：

bmrlab / gendam