Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[WIP] enabling large tensor support for binary broadcast operators #16714

Closed
wants to merge 1 commit into from
Closed

[WIP] enabling large tensor support for binary broadcast operators #16714

wants to merge 1 commit into from

Conversation

access2rohit
Copy link
Contributor

Description

enabling int64 for binary broadcast ops:
TOPIOp
_contrib_tvm_vadd
numpy binary logic ops
_npi_arctan2
_npi_hypot

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:
  • Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
  • Code is well-documented:
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Testing

To be updated

@ChaiBapchya ChaiBapchya mentioned this pull request Nov 7, 2019
Merged
6 tasks
apeforest
apeforest suggested changes Jan 9, 2020
View reviewed changes
Copy link
Contributor

@apeforest apeforest left a comment

Choose a reason for hiding this comment

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

blocking this PR because it's outdated

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

@apeforest apeforest apeforest requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@access2rohit @apeforest