Huazuo Gao
Huazuo Gao
可以解决首次登录时的-6问题, 但是cookies过期后有其他问题 ``` Traceback (most recent call last): File "/usr/local/lib/python2.7/dist-packages/fuse.py", line 495, in _wrapper return func(*args, **kwargs) or 0 File "/usr/local/lib/python2.7/dist-packages/fuse.py", line 693, in readdir fip.contents.fh): File "baidufuse.py", line 165,...
`.username.cookies` 没有保存domain, 重新登录之后导致冲突. 具体来说是`self.session.cookies['BDUSS']` 会抛异常. https://github.com/gaohuazuo/baidupcsapi/commit/b3b34c27dd8809ab2f3e0526e7e7d39a6bc71573 https://github.com/gaohuazuo/baidupcsapi/commit/ba3a4dcf364bc13a5181c7cf8185ca43cd7afc2b 此外 `BaseClass._request` 出错后只是重新登录, 没有重试, 不知是算feature还是bug. 这种情况下baidu-fuse没有检查.
没有提PR是因为 https://github.com/gaohuazuo/baidupcsapi/commit/eaac7a3d015548caa377565b37da273cf473f5dc 写得比较丑, 只能算是一个workaround.
我认为应该检查参数是否为 iterable of tensors,而不是允许 nested iterable of tensors 设计语法或者 API 的时候,功能更强并不一定是好事。用尽可能短的代码实现尽可能多的功能并不是全部的追求,尽可能早地检查出尽可能多的错误也不能忽视。从 #171 的讨论看,nested iterable of tensors 目前还缺乏实际的 usecase,现在添加这个功能只能期望未来有正面效果,但却会立即产生减少错误检查的负面效果。 有时候,宽松的设计不是为了实现更多功能,而是为了容许一些无害的错误,例如 `attach(tensor)` 可以认为是无害的错误,也正因如此现有的设计也容许了这种用法。但像 `attach([module.parameter()])` 算不算无害的错误我认为还有待讨论,即使算无害的错误,原则上也不能直接扩展到 nested iterable of tensors。当然如果特例变多,确实应该归纳到一种更统一的框架下。不过也是从美学角度考虑,list of tensors 已经是自动微分最自然的抽象,改成...
从报错看是 resize backward 算子的问题,但我们目前未能复现此问题。不过可以确认 input 或 output 的 height / width 中含有 0 时会产生类似的报错。