Skip to content

Conversation

@IvanKobzarev
Copy link
Contributor

@IvanKobzarev IvanKobzarev commented Sep 9, 2024

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 9, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/135501

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 1 Unrelated Failure

As of commit d96cb82 with merge base dfb2b66 (image):

NEW FAILURE - The following job has failed:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@ezyang ezyang requested review from bdhirsh and removed request for ezyang September 10, 2024 12:10

# Key to use for AOTAutogradCache
cache_key: Optional[str] = None
profile_duration: bool = os.environ.get("PYTORCH_AOTD_DEBUG_PROFILE", "0") == "1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't it be better for this to be controlled in torch/_functorch/config.py? Rather than randomly scattered here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, yes, did not know about functorch/config.


def get_events() -> List[Event]:
global events
return events
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of handrolling your own thing, you should use @jamesjwu's profiler, which also has integrated support with tlparse and scuba. #134118

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. Mostly did it for debug profiling, we can ignore this PR. I will start using James's profiler when start working on subclasses compilation perf.

@IvanKobzarev
Copy link
Contributor Author

Abandoning in favor of #136478

injiiiiil pushed a commit to injiiiiil/654 that referenced this pull request Oct 1, 2024
ghstack-source-id: 63e440b
Pull Request resolved: pytorch/pytorch#135501
@github-actions github-actions bot deleted the gh/IvanKobzarev/68/head branch October 27, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants