search

DIYer22 / boxx

Tool-box for efficient build and debug in Python. Especially for Scientific Computing and Computer Vision.
492 stars 39 forks source link

Update ylimgTool.py #16

Closed qianyizhang closed 1 year ago

qianyizhang commented 1 year ago

add titles to showImgLists

qianyizhang commented 1 year ago

hello? i'd rather you accept my MR than mantaining my own fork

DIYer22 commented 1 year ago

我看你会中文, 我就中文回复了. 首先感谢贡献. 原本打算我在你的基础上新增支持 dict 的, 就是最近 chatGPT 之类的太火了, 导致事情比较多,分散了注意力, 就拖着了, 实在抱歉! 我提几点建议:

  1. titles 除了从titles = kv.pop("titles",[]) 也可以自动检测 imgs 是 dict, 把 dict 的 keys 作为 titles
  2. 为了 boxx 支持老版本的 Python, 不推荐添加 lint, 比如: split: int

如果你方便的话,可以你来改. 如果你不方便的话, 我再找时间改一改. 然后就 merge

qianyizhang commented 1 year ago

你这2点是冲突的：

我的usecase：逐帧展示video结果，title加些信息

qianyizhang commented 1 year ago

我的建议是，drop 老版本支持。。现在pytorch2 都3.7还是3.8起步了。

DIYer22 commented 1 year ago

由于 boxx 开发于 2017 年, 那时候 Python2 还很流行, 做了很多兼容操作, 架构也不科学, 导致其积重难返了. 我正在规划一个新的项目(暂时叫 mxx), 会面向现代的 Python, 对 boxx 做简化, 重构和拆分. 到时候对 boxx 的期待就是在老的系统和代码上能用.