megemini

Results 108 comments of megemini

另外,关于 tensor.pyi.in 这个模板文件的维护问题,我的想法是,不需要特殊维护,因为,后面 CI 会检查 api 的 typing,如果 tensor 接口有变,且没有办法自动生成有效的签名,那么,如果不修改 tensor.pyi.in 文件手动添加,则 CI 检查 `可能` 会 fail(没有示例代码和类型测试用例,那就没办法检查了 ... ...)。 这个可以在 《Paddle 中的类型提示》 的开发文档里面写明 ~

抱歉,才发现有几个问题没有回复 ~~~ > 应该是太多了搜索卡住了,但 `__ror__` 应该是没有的 我看了一下,dir(paddle.Tensor) 里面确实没有 ~ 你这个 `__ror__` 是哪来的? > 诶?为啥不从源头删掉?这个文件应该不需要 mypy 检查的 tensor.pyi.in 需要检查啊,不然怎么保证开发者修改这个文件的时候没有改出问题? 另外,对比 `https://github.com/cattidea/paddlepaddle-stubs/blob/main/paddle-stubs/_typing/tensor.pyi`,以下方法不在 paddle.Tensor 中,进而也不在此次生成的 stub 文件中: ``` __pos__ __rfloordiv__ __rmod__ __rmatmul__ __ror__...

> 包括 `__ror__`,这些应该是我觉得「应该」存在的方法,不过这里应该和运行时保持一致,这些不存在的就不要加了 > OK ~ > > tensor.pyi.in 需要检查啊,不然怎么保证开发者修改这个文件的时候没有改出问题? > > 检查方式有想好么?可以考虑复用 `pre-commit`,不建议在 CI 单独开逻辑,会和开发工作流割裂,提完 PR 在 CI 上才发现问题,不了解的同学调试起来也很麻烦 嗯!已经改为 `tensor.prototype.pyi` ,`pre-commit` 可以直接检查 ~~~ 另外,做以下修改: - setup 里面只打包 `tensor.pyi`,也就是生成的...

### Update 20240523: - _typing.basic.py 添加 TensorLike - core.pyi 添加 Place - 更新 tensor.prototype.pyi

> * 添加了 framework/**init**.pyi 可以不用添加这个吧? ```paddle.__init__.py``` 里面已经有了 ~ > * 在setup.py里添加了相关打包内容,但是我编译后安装并没有打包成功 有两个地方要改, - python/setup.py.in - setup.py https://github.com/PaddlePaddle/Paddle/pull/63901

> 我来改一下这个 PR,尽快推动本 PR 合入,以免阻塞整个项目进度 还有 https://github.com/PaddlePaddle/Paddle/pull/63953 没 review,是不是漏掉了 😂

### Update 20240510 - mypy 的配置放到 `pyproject.toml` 中 - mypy 的 `cache dir` 使用绝对路径 - 去掉 `--debug` 参数,减少日志输出(不过,由于 CI 日志的特殊性,仍然会输出两次 error 日志) - 根据 PR 的 title 判断是否进行 type checking...

### Update 20240529 - `paddle_build.sh` 中将 `exec_samplecode_test` 和 `exec_type_checking` 统一放到新增的 `exec_samplecode_checking` 中 - 检查标题统一改为 `typing` 和 `typing all` - 修改 `type_checking.py` 中的路径表示 另外: > 示例代码和类型检查是否可以分别抽一个函数呢?CI 入口一般只做分发,不适合写太多逻辑~ > 啊,不对,我才发现是上面的 `run_type_checking` 的语义不对,上面的...

### Update 20240510 - 延长 radam / nadam 的单测时间 - 不使用 uci_housing 数据,使用 random 数据进行单测 (uci_housing 可能出现超时等情况,特使用随机数据代替)。 另外,PR-CI-Coverage 中 radam / nadam 是通过的了,fail 的 api 应该不涉及这两个地方 ~ @cxxly 请评审 ~

### Update 20240514 - 修改 docstring @sunzhongkai588 请评审 ~