Skip to content

Added floordiv simplification rule needed for models#85768

Closed
Chillee wants to merge 1 commit intogh/chillee/130/basefrom
gh/chillee/130/head
Closed

Added floordiv simplification rule needed for models#85768
Chillee wants to merge 1 commit intogh/chillee/130/basefrom
gh/chillee/130/head

Conversation

@Chillee
Copy link
Copy Markdown
Collaborator

@Chillee Chillee commented Sep 27, 2022

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Sep 27, 2022

🔗 Helpful Links

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

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

✅ No Failures

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

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

@ezyang
Copy link
Copy Markdown
Contributor

ezyang commented Sep 27, 2022

For reference, how did you figure it out?

@Chillee
Copy link
Copy Markdown
Collaborator Author

Chillee commented Sep 27, 2022

@ezyang I saw all the time was being spent in solve, and I looked at the expressions we were solving, and they all looked much more complicated than I remembered them being (and also heavily involved FloorDiv).

I took a closer look at one of them, and realized that the numerator was something like a * b * c and denominator was b *c * 5, which should be simplified.

@ezyang
Copy link
Copy Markdown
Contributor

ezyang commented Sep 27, 2022

Seems like we should record stats on expression complexity so we can see if this sort of thing is happening

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants