Skip to content

Conversation

@eellison
Copy link
Contributor

@eellison eellison commented Apr 1, 2022

Stack from ghstack:

  • Add option to only run some graphs
  • Add NNC Static vs Dynamic
  • Update make_tensor bc it wasnt using strides

Differential Revision: D35374000

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

facebook-github-bot commented Apr 1, 2022

🔗 Helpful links

💊 CI failures summary and remediations

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


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


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Apr 1, 2022
@eellison eellison requested a review from davidberard98 April 1, 2022 15:33
- Add option to only run some graphs
- Add NNC Static vs Dynamic 
- Update make_tensor bc it wasnt using strides

[ghstack-poisoned]
eellison pushed a commit that referenced this pull request Apr 1, 2022
ghstack-source-id: 3fc71ff
Pull Request resolved: #75108
Copy link
Contributor

@davidberard98 davidberard98 left a comment

Choose a reason for hiding this comment

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

looks good!

- Add option to only run some graphs
- Add NNC Static vs Dynamic 
- Update make_tensor bc it wasnt using strides

[ghstack-poisoned]
@eellison
Copy link
Contributor Author

eellison commented Apr 4, 2022

@eellison has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

- Add option to only run some graphs
- Add NNC Static vs Dynamic 
- Update make_tensor bc it wasnt using strides

Differential Revision: [D35374000](https://our.internmc.facebook.com/intern/diff/D35374000)

[ghstack-poisoned]
@eellison
Copy link
Contributor Author

eellison commented Apr 5, 2022

@eellison has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Apr 6, 2022
Summary:
Pull Request resolved: #75108

- Add option to only run some graphs
- Add NNC Static vs Dynamic
- Update make_tensor bc it wasnt using strides

Test Plan: Imported from OSS

Reviewed By: ejguan

Differential Revision: D35374000

Pulled By: eellison

fbshipit-source-id: df16b8647f2309a8837207cacba55d30f46845ce
@facebook-github-bot facebook-github-bot deleted the gh/eellison/289/head branch April 10, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants