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

Remove access to partially filled PathsAndConsumes object #1194

Open
wddgit opened this issue Jan 27, 2025 · 0 comments
Open

Remove access to partially filled PathsAndConsumes object #1194

wddgit opened this issue Jan 27, 2025 · 0 comments
Assignees
Labels

Comments

@wddgit
Copy link
Collaborator

wddgit commented Jan 27, 2025

This involves moving Services using the preBeginJob signal to use the LookupInitializationComplete signal. Here is the list of relevant services: NVProfilerService, ProcessCallGraph, FastTimerService, DependencyGraph. Double check to make sure that is a complete list. And when done remove the PathsAndConsumes argument from the PreBeginJob signal if possible.

Probably this can be done in 1 PR but split this up into multiple if it turns out to be more difficult than expected.

@wddgit wddgit added the Task label Jan 27, 2025
@wddgit wddgit self-assigned this Jan 27, 2025
@github-project-automation github-project-automation bot moved this to 🔖 Ready in Framework tasks Jan 27, 2025
@wddgit wddgit moved this from 🔖 Ready to 🏗 In progress in Framework tasks Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🏗 In progress
Development

No branches or pull requests

1 participant