Skip to content

moving vector-add to new repo.#7

Merged
JoeOster merged 2 commits intooneapi-src:masterfrom
Jenn500:master
Jul 16, 2020
Merged

moving vector-add to new repo.#7
JoeOster merged 2 commits intooneapi-src:masterfrom
Jenn500:master

Conversation

@Jenn500
Copy link
Copy Markdown
Contributor

@Jenn500 Jenn500 commented Jun 30, 2020

moving the latest vector-add to new repo, updated sample.json, added vector-add-usm.vcxproj
Signed-off-by: jennifer

Signed-off-by: jennifer.l.jiang <jljiang@orcsle165.jf.intel.com>
@anjgola anjgola requested review from mkitez and pmpeter1 June 30, 2020 23:07
anjgola
anjgola previously approved these changes Jun 30, 2020
@anjgola
Copy link
Copy Markdown
Contributor

anjgola commented Jun 30, 2020

@mkitez did CI run on this? @Jenn500 I am not sure if it did so let us wait until Roman confirms before merging it.

@Jenn500
Copy link
Copy Markdown
Contributor Author

Jenn500 commented Jul 1, 2020

all checks are failed. But could not find the detail on why.

{"id": "cpu_usm","build": ["nmake -f Makefile.win build_usm"],"run": ["nmake -f Makefile.win run_usm"],"clean": ["nmake -f Makefile.win clean"]},
{"id": "fpga_emu_buffers","build": ["nmake -f Makefile.win.fpga clean","nmake -f Makefile.win.fpga"],"run": ["nmake -f Makefile.win.fpga run"]}
]
}
Copy link
Copy Markdown
Contributor

@mkitez mkitez Jul 2, 2020

Choose a reason for hiding this comment

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

The existing configurations are the correct ones in terms of syntax. Please, review them and make required updates if needed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll make the change and resubmit.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mkitez Jennifer is on vacation, can you help me understand what the required updates are?

Copy link
Copy Markdown
Contributor

@anjgola anjgola Jul 15, 2020

Choose a reason for hiding this comment

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

@sravanikonda The json field's value in this PR for 'ciTests' is outdated. It is not using the new format specified here : https://github.com/oneapi-src/oneAPI-samples/wiki/sample-json-ci-test-object

Copy link
Copy Markdown
Contributor

@JoeOster JoeOster left a comment

Choose a reason for hiding this comment

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

There is no license.txt file

@Jenn500
Copy link
Copy Markdown
Contributor Author

Jenn500 commented Jul 16, 2020

I've updated the sample.json. please check if it's correct.

@Jenn500 Jenn500 requested review from JoeOster and mkitez July 16, 2020 18:49
@JoeOster JoeOster merged commit b937ac3 into oneapi-src:master Jul 16, 2020
jimmytwei pushed a commit to jimmytwei/oneAPI-samples that referenced this pull request Mar 29, 2023
Changed data type of 'triad_bytes' from int to size_t to prevent an integer overflow
jimmytwei pushed a commit that referenced this pull request Jun 6, 2025
Sticking to specific tensorflow and pandas version.
Modifying some notebook cells to perform generic actions.
Using specific commit url when retrieving the pretrained_word_embeddings.py file.

Signed-off-by: Mora Jimenez, Kevin <kevin.mora.jimenez@intel.com>
Co-authored-by: Mora Jimenez, Kevin <kevin.mora.jimenez@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants