Skip to content

Delete obsolete message in Dict: "or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"#65164

Closed
pbelevich wants to merge 5 commits intogh/pbelevich/158/basefrom
gh/pbelevich/158/head
Closed

Delete obsolete message in Dict: "or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"#65164
pbelevich wants to merge 5 commits intogh/pbelevich/158/basefrom
gh/pbelevich/158/head

Conversation

@pbelevich
Copy link
Contributor

@pbelevich pbelevich commented Sep 16, 2021

…:GenericDict(c10::impl::deprecatedUntypedDict())

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

facebook-github-bot commented Sep 16, 2021

🔗 Helpful links

💊 CI failures summary and remediations

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


  • 2/2 failures introduced in this PR

2 failures not recognized by patterns:

Job Step Action
GitHub Actions linux-bionic-py3.6-clang9 / build Unknown 🔁 rerun
GitHub Actions linux-xenial-cuda11.3-py3.6-gcc7 / build Unknown 🔁 rerun

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 to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@pbelevich pbelevich changed the title Delete obsolete message: or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict()) [JIT] Delete obsolete message: or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict()) Sep 16, 2021
@pbelevich pbelevich requested a review from zdevito September 16, 2021 23:35
…o, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"

Looks like it was forgotten in #25439




[ghstack-poisoned]
…o, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"

Looks like it was forgotten in #25439




[ghstack-poisoned]
…o, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"

Looks like it was forgotten in #25439




[ghstack-poisoned]
Copy link
Contributor

@mrshenli mrshenli left a comment

Choose a reason for hiding this comment

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

LGTM

…o, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())"

Looks like it was forgotten in #25439




[ghstack-poisoned]
@pbelevich
Copy link
Contributor Author

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

@facebook-github-bot
Copy link
Contributor

@pbelevich merged this pull request in 5826d20.

@facebook-github-bot facebook-github-bot deleted the gh/pbelevich/158/head branch September 24, 2021 14:17
@tugsbayasgalan tugsbayasgalan changed the title [JIT] Delete obsolete message: or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict()) Delete obsolete message in Dict: "or if you absolutely have to, use c10::impl::GenericDict(c10::impl::deprecatedUntypedDict())" Oct 15, 2021
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