Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CodeStyle] Legacy python code cleanup tracking issue #46837

Closed
SigureMo opened this issue Oct 10, 2022 · 5 comments
Closed

[CodeStyle] Legacy python code cleanup tracking issue #46837

SigureMo opened this issue Oct 10, 2022 · 5 comments
Assignees
Labels
PFCC Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc status/close 已关闭

Comments

@SigureMo
Copy link
Member

SigureMo commented Oct 10, 2022

项目说明

用于跟踪 Call-for-Contributions: Python 2.7 相关代码退场Python 3.5/3.6 相关代码退场
各个任务的计划和进度

成果总结

✅ Python 2.7 退场(8/8)

子任务 具体内容和完成时间 相关 PR 修复行数 作者
✅ 1. 删除python2子包 ✅ 1.1 移除 __future__ import(2022/09/28) #46411, #46463, #46466, #46736 3613 SigureMo, Yulv-git
✅ 1.2 移除six(2022/11/01) #46965, #47334 906 SigureMo
✅ 2. 删除没有其它功能的 Python 2 模块 ✅ 2.1 移除 类中不必要的显式 object 继承(2022/11/08) #47642 #47730 524 SigureMo
✅ 2.2 移除 super() 函数中不必要的参数(2022/11/22) #47549, #47643 1900 SigureMo
✅ 2.3 移除 compat.py 文件(2022/10/25) #46686, #46923, #47035, #47036, #47097, #47269 836 SigureMo
✅ 3. 清理 Python2 相关逻辑分支 ✅ 3. 清理 Python2 相关逻辑分支(2022/10/11) #46696 92 caolonghao
✅ 4. 删除非必要的环境依赖(Python2) ✅ 4. 删除非必要的环境依赖(Python2)(2023/01/03) #48685 51 SigureMo
✅ 5. 清理文档中涉及 Python 2 的内容 ✅ 5. 清理文档中涉及 Python 2 的内容(2022/12/01) docs#5466 125 caolonghao

Python 3.5/3.6 退场(3/3)

子任务和完成时间 相关PR 修复行数 作者
✅ 1. 清理低于 Python 3.7 相关逻辑分支(2022/11/29) #48426, #48540 78 gsq7474741
✅ 2. 删除非必要的环境依赖(Python 3.5/3.6)(2023/2/6) #49558 125 gsq7474741
✅ 3. 清理文档中涉及 Python 3.5/3.6 的内容(2022/12/01) docs#5460 65 gsq7474741

其他成果总结

子任务和完成时间 相关 PR 修复行数 作者
✅ 1. 使用 open 替换 io.open(2022/10/11) #46751 18 SigureMo
✅ 2. 清理为兼容 Python2 而增加的 next 方法(2022/11/08) #47728 17 SigureMo
✅ 3. 替换 metaclass 旧的声明语法(2022/11/09) #47730 6 SigureMo
✅ 4. 移除字符串字面量无用的 u 前缀(2022/11/09) #47727 64 SigureMo
✅ 5. 使用 inspect.getfullargspec 替换 inspect.getargspec(2022/11/22) #48218 43 SigureMo
✅ 6. 移除 DeprecationWarning filter(2022/12/05) #48639 8 gsq7474741
✅ 7. 更新 PyPI README 文档 (2022/12/28) #48640 42 gsq7474741
✅ 8. 清理弃用的 typing.Text(2023/01/04) #49506 6 SigureMo
✅ 9. 清理低于 Python 3.7 的依赖项、更新 setup.py classifiers 字段(2023/01/04) #49516 10 gsq7474741

Note

这里仅统计已经完成的项目,部分正在进行/将要进行的见下面 Comment,如有遗漏,可在下面直接回复或者在该 PR 里 @SigureMo 一下~

@paddle-bot

This comment was marked as off-topic.

@paddle-bot paddle-bot bot added the PFCC Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc label Oct 10, 2022
@SigureMo SigureMo assigned SigureMo and unassigned levi131 Oct 10, 2022
@SigureMo
Copy link
Member Author

SigureMo commented Oct 10, 2022

Python2、Python3.5、Python3.6 遗留代码退场计划各个子任务的计划和进度

✅ remove future imports [py2]

✅ Python2 条件分支 [py2]

✅ 使用 open 替换 io.open [py2]

✅ 移除 paddle.compat 模块 [py2]

✅ 删除第三方包 six 的使用及依赖 [py2]

✅ 清理显式继承 object [py2]

✅ 清理 super() 继承时不必要的参数 [py2]

✅ 替换 __metaclass__ 的元类声明方式 [py2]

✅ 清理 u 前缀的字符串 [py2]

✅ 清理 Python2 next 方法 [py2]

✅ 使用 inspect.getfullargspec 替换 inspect.getargspec [py2]

✅ 低于 Python 3.7 条件分支清理 [py36-]

✅ 移除 DeprecationWarning filter [py36-]

✅ 移除为 Python 3.2-3.6 做的 DeprecationWarning 兼容性处理 [py36-]

✅ 清理不必要的 Python2 相关注释 [py2]

✅ 更新 PyPI README 文档 [py36-]

✅ 清理 Docker 镜像中不必要的 python2 安装 [py2]

✅ 清理低于 Python 3.7 的依赖项、更新 setup.py classifiers 字段 [py36-]

✅ 清理弃用的 typing.Text [py36-]

✅ 清理弃用的 unittest 的 aliases [py36-]

✅ 清理 Docker 镜像中不必要的 python 3.5/3.6 安装 [py36-]

清理文档中的相关使用

其他...

TODO...

@luotao1 luotao1 self-assigned this Oct 10, 2022
@ouzejun
Copy link

ouzejun commented Nov 20, 2022

@SigureMo 您好,我想做任务2.2移除super()函数中不必要的参数

@SigureMo
Copy link
Member Author

@SigureMo 您好,我想做任务2.2移除super()函数中不必要的参数

emmm,这个的话其实已经基本上做完了,上面的 comment 记录了详情(不是最上面那个),剩下几个在示例代码中的在 #47643 修改了还没合入,此外就只有几个增量了~

感兴趣可以移除下增量,以及 Markdown 里还有一些没有移除的,「清理显式继承 object」也有一些可以一起完成(剩余的很少,总计估计也就 10~20 个吧)

@ouzejun
Copy link

ouzejun commented Nov 20, 2022

哦好的,那我看看其他合适些的代码

@SigureMo SigureMo changed the title [CodeStyle] Legacy python2 tracking issue [CodeStyle] Legacy python code cleanup tracking issue Nov 22, 2022
@luotao1 luotao1 moved this to In Progress in Call for Contributions Dec 6, 2022
@luotao1 luotao1 closed this as completed Feb 6, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Call for Contributions Feb 6, 2023
@paddle-bot paddle-bot bot added the status/close 已关闭 label Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PFCC Paddle Framework Contributor Club,https://github.com/PaddlePaddle/community/tree/master/pfcc status/close 已关闭
Projects
Development

No branches or pull requests

4 participants