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

enhance: support run analyzer #2622

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aoiasd
Copy link
Contributor

@aoiasd aoiasd commented Feb 8, 2025

@mergify mergify bot added the dco-passed label Feb 8, 2025
@mergify mergify bot added the ci-passed label Feb 18, 2025
@aoiasd aoiasd force-pushed the master branch 2 times, most recently from 53851ac to 4796d39 Compare February 18, 2025 07:47
@aoiasd aoiasd changed the title support run analyzer enhance: support run analyzer Feb 18, 2025
@XuanYang-cn XuanYang-cn added the PR | need to cherry-pick to 2.x This PR need to be cherry-picked to 2.x branch label Feb 18, 2025
@XuanYang-cn XuanYang-cn added this to the 2.6.0 milestone Feb 18, 2025
@aoiasd aoiasd force-pushed the master branch 3 times, most recently from abe3ccf to f7a6367 Compare February 18, 2025 10:26
@XuanYang-cn
Copy link
Contributor

/lgtm

@sre-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aoiasd
To complete the pull request process, please ask for approval from xuanyang-cn after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

self.tokens = info.tokens

def __str__(self) -> str:
return f"{{tokens: {self.tokens}}}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this double {{}} output as designed?

{{tokens: xxxx}}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

{{ means '{' in f"",
img_v3_02jl_ef184b3a-3a37-46a5-a130-ebd2dd218a2g

Signed-off-by: aoiasd <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-passed dco-passed PR | need to cherry-pick to 2.x This PR need to be cherry-picked to 2.x branch size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants