Skip to content

Conversation

ezyang
Copy link
Contributor

@ezyang ezyang commented Dec 15, 2022

…ders

Signed-off-by: Edward Z. Yang <ezyang@fb.com>

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Dec 15, 2022

🔗 Helpful Links

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

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

✅ No Failures

As of commit 1615c8f:
💚 Looks good so far! There are no failures yet. 💚

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

@pytorch-bot pytorch-bot bot added the release notes: fx release notes category label Dec 15, 2022
ezyang added a commit that referenced this pull request Dec 15, 2022
…ders

Signed-off-by: Edward Z. Yang <ezyangfb.com>

ghstack-source-id: 8c09e3c
Pull Request resolved: #90912
…to placeholders"

Signed-off-by: Edward Z. Yang <ezyangfb.com>

[ghstack-poisoned]
@@ -768,6 +771,43 @@ def evaluate_guards_for_args(self, placeholders, args):
code = self.codegen_guards(placeholders, arg_names)
return eval(code, {}, dict(zip(arg_names, args)))

def bind_symbols(self, placeholders, args):
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you add a small doc explaining what this does?

…to placeholders"

Signed-off-by: Edward Z. Yang <ezyangfb.com>

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 67436f6.

@facebook-github-bot facebook-github-bot deleted the gh/ezyang/1661/head branch June 8, 2023 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged release notes: composability release notes category release notes: fx release notes category topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants