Skip to content

[FlowInsight] flame graph fix time duration for running tasks#556

Merged
xsuler merged 3 commits intomainfrom
dev-flame-st
Mar 31, 2025
Merged

[FlowInsight] flame graph fix time duration for running tasks#556
xsuler merged 3 commits intomainfrom
dev-flame-st

Conversation

@xsuler
Copy link
Copy Markdown
Collaborator

@xsuler xsuler commented Mar 31, 2025

Why are these changes needed?

flame graph fix time duration for running tasks

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Signed-off-by: sule <sule.sl@antgroup.com>
@xsuler xsuler requested a review from wumuzi520 March 31, 2025 10:10
@xsuler xsuler self-assigned this Mar 31, 2025
Signed-off-by: sule <sule.sl@antgroup.com>
Copy link
Copy Markdown
Collaborator

@wumuzi520 wumuzi520 left a comment

Choose a reason for hiding this comment

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

LGTM

@xsuler xsuler merged commit a3e8f51 into main Mar 31, 2025
4 of 5 checks passed
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