-
Notifications
You must be signed in to change notification settings - Fork 364
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
fix: change profiler docs url #9034
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9034 +/- ##
==========================================
- Coverage 47.80% 41.73% -6.08%
==========================================
Files 1161 841 -320
Lines 143646 104178 -39468
Branches 2373 2373
==========================================
- Hits 68673 43475 -25198
+ Misses 74820 60550 -14270
Partials 153 153
Flags with carried forward coverage won't be shown. Click here to find out more.
|
1f08a3c
to
7dd2578
Compare
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change looks good to me, but I think it maybe better to land this after the docs change lands.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
close this pr because profiler page doesnt require doc link |
Description
Request from @tara-det-ai
The original docs change PR: #9011, and #9032
WEB-2004
Test Plan
/model-dev-guide/profiling.html
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket