Skip to content

fix missing comma in activation benchmarks#35104

Closed
vkuzo wants to merge 1 commit intogh/vkuzo/14/basefrom
gh/vkuzo/14/head
Closed

fix missing comma in activation benchmarks#35104
vkuzo wants to merge 1 commit intogh/vkuzo/14/basefrom
gh/vkuzo/14/head

Conversation

@vkuzo
Copy link
Copy Markdown
Contributor

@vkuzo vkuzo commented Mar 20, 2020

Stack from ghstack:

Summary:

I missed this in #34959
after a rebase, fixing.

Test Plan:

running benchmarks no longer crashes
CI

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D20560908

Summary:

I missed this in #34959
after a rebase, fixing.

Test Plan:

running benchmarks no longer crashes
CI

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Mar 20, 2020
Summary:

I missed this in #34959
after a rebase, fixing.

Test Plan:

running benchmarks no longer crashes
CI

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 4b04e68
Pull Request resolved: #35104
Copy link
Copy Markdown
Contributor

@hl475 hl475 left a comment

Choose a reason for hiding this comment

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

Thanks!

@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Mar 20, 2020

💊 CircleCI build failures summary and remediations

As of commit 9d32d4c (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no CircleCI failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

This comment has been revised 1 time.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in bf41a76.

1 similar comment
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in bf41a76.

@facebook-github-bot facebook-github-bot deleted the gh/vkuzo/14/head branch March 24, 2020 14:16
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#35104

I missed this in pytorch#34959
after a rebase, fixing.

Test Plan:
running benchmarks no longer crashes
CI

Imported from OSS

Differential Revision: D20560908

fbshipit-source-id: a5494e23953d3c9007e9874d673896291b5322e0
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.

4 participants