Skip to content

Conversation

@suo
Copy link
Member

@suo suo commented Aug 3, 2018

Support a few basic operators:

  • eq
  • add
  • len
  • select (indexing)

@suo suo added the oncall: jit Add this issue/PR to JIT oncall triage queue label Aug 3, 2018
@suo suo requested a review from jamesr66a August 3, 2018 22:07
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

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

c = a + b
return c == [1, 2, 3]

self.checkScript(test_list_add_empty, (), optimize=True)

This comment was marked as off-topic.

This comment was marked as off-topic.

for (size_t i = 0; i < a->elements().size(); ++i) {
const auto& a_element = a->elements()[i];
const auto& b_element = b->elements()[i];
if (!a_element.equal(b_element)) {

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@suo suo force-pushed the pr/list_concat branch from 41605b5 to 44a8c75 Compare August 6, 2018 07:05
@suo suo changed the title [jit][script] Support list concatenation and equality operators [jit][script] Support some basic list operations Aug 7, 2018
Copy link
Contributor

@zdevito zdevito left a comment

Choose a reason for hiding this comment

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

These look good!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

michaelsuo is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@apaszke apaszke deleted the pr/list_concat branch August 7, 2018 14:04
->asValue(loc, method);

} else {
// if it's a single tensor, map tensor[idx] -> tensor.select(0, idx)

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Support a few basic operators:
- eq
- add
- len
- select (indexing)
Pull Request resolved: pytorch#10225

Differential Revision: D9172338

Pulled By: michaelsuo

fbshipit-source-id: 6e75ec1453b9589b0fb4698598ecdba5a5fccff9
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants