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

添加一个基础的变量追踪机制 #54

Closed
wants to merge 7 commits into from

Conversation

zrr1999
Copy link
Contributor

@zrr1999 zrr1999 commented May 22, 2023

提供了一个 任务三 中提到的类 VariableTracker 机制的一个基础版。

args: list[VariableBase],
kwargs: dict[str, VariableBase],
):
assert isinstance(fn, CallableVariable | ObjectVariable)
Copy link
Collaborator

@SigureMo SigureMo May 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个我记得 Python 3.10 才能用的吧,而且 Python 3.10 下性能远低于原来的写法,建议至少 3.11 再用(虽然还是慢……)

python/cpython#91603

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好的,我去改一下

zrr1999 and others added 4 commits June 15, 2023 09:58
commit f5faff6
Author: Allen Guo <guosonglin001@gmail.com>
Date:   Sat Jun 10 15:02:16 2023 +0800

    add PyCodegen demo (PFCCLab#57)

commit 4e1f6ee
Author: co63oc <co63oc@users.noreply.github.com>
Date:   Sun May 28 14:51:07 2023 +0800

    fix a typo in interpreter.cc (PFCCLab#55)

commit 4caa871
Author: co63oc <co63oc@users.noreply.github.com>
Date:   Sun May 28 14:50:29 2023 +0800

    fix a typo in fx_profiling.py (PFCCLab#56)
@zrr1999 zrr1999 marked this pull request as ready for review June 15, 2023 10:02
@zrr1999 zrr1999 changed the title [WIP] 添加一个基础的变量追踪机制 添加一个基础的变量追踪机制 Jun 15, 2023
@zrr1999 zrr1999 requested a review from SigureMo June 15, 2023 10:30
@zrr1999 zrr1999 mentioned this pull request Sep 3, 2023
@zrr1999 zrr1999 closed this Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants