openvino icon indicating copy to clipboard operation
openvino copied to clipboard

[Good First Issue]: Enable Tensor.copyTo() method

Open almilosz opened this issue 9 months ago • 21 comments

Context

OpenVINO works in Node.js environment! We are looking for new contributors who can help with enabling C++ API methods in JavaScript side.

First of all read Node.js API Contribution Guide.

Task Details

Expose ov::Tensor::copy_to() method. From JavaScript side it will be Tensor.copyTo() method.

Useful links

What needs to be done?

  • Implement the method on C++ side (src/bindings/js/node/src/tensor.cpp)
  • Update TypeScript definitions in ./src/bindings/js/node/lib/addon.ts
  • Create unit test for added functionality using Node.js Test Runner.

How to take this issue

To take this issue leave text: .take as the comment in this issue. It will assign this issue to you automatically. Please, make sure that the issue has the status Contributors Needed and doesn't have another user in Assingnees.

Example Pull Requests

  • https://github.com/openvinotoolkit/openvino/pull/23743
  • https://github.com/openvinotoolkit/openvino/pull/24023

Resources

Contact points

@almilosz @Retribution98

Ticket

almilosz avatar Mar 26 '25 15:03 almilosz

.take

Daves1245 avatar Mar 26 '25 20:03 Daves1245

Thank you for looking into this issue! Please let us know if you have any questions or require any help.

github-actions[bot] avatar Mar 26 '25 20:03 github-actions[bot]

hey @Daves1245 this ticket has been taken and PR delivered by @sanleo-wq focus on 2nd one you assigned for yourself, please

mlukasze avatar Mar 27 '25 05:03 mlukasze

@sanleo-wq please, assign this ticket to yourself using comment with content: .<remove the space>take, like here: https://github.com/openvinotoolkit/openvino/issues/29758#issuecomment-2755715499

mlukasze avatar Mar 27 '25 05:03 mlukasze

.take

sanleo-wq avatar Mar 27 '25 09:03 sanleo-wq

Thank you for looking into this issue! Please let us know if you have any questions or require any help.

github-actions[bot] avatar Mar 27 '25 09:03 github-actions[bot]

.take

lsanleo avatar Mar 31 '25 08:03 lsanleo

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

github-actions[bot] avatar Mar 31 '25 08:03 github-actions[bot]

.take

somilshivhare avatar Apr 02 '25 19:04 somilshivhare

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

github-actions[bot] avatar Apr 02 '25 19:04 github-actions[bot]

Hi @Daves1245 ,are you still working on this issue? If not, I would love to take over and contribute. Let me know, thanks!

somilshivhare avatar Apr 02 '25 19:04 somilshivhare

Hi @somilshivhare and @Daves1245 Let me know if you start working on this issue, I would love to help and contribute.

Shruti2301 avatar Apr 04 '25 04:04 Shruti2301

@Daves1245 please let me know if we can contrib ute together on this isse .Please ket me know through the reply asap.

somilshivhare avatar Apr 04 '25 04:04 somilshivhare

This task is open now. To take this issue leave text: .take as the comment in this issue. Please, make sure that the issue has the status Contributors Needed and doesn't have another user in Assingnees.

almilosz avatar May 19 '25 06:05 almilosz

.take

somilshivhare avatar May 19 '25 07:05 somilshivhare

Hi @almilosz and team,

Thank you for reopening this task. I have now claimed it by commenting .take, and I’m excited to start working on implementing the Tensor.copyTo() method.

I'll follow the Node.js API Contribution Guide and refer to the resources and example PRs shared. Please let me know if there are any specific expectations or tips before I begin.

Looking forward to contributing!

Best regards, Somil Shivhare

On Mon, 19 May 2025 at 12:27, Alicja Miloszewska @.***> wrote:

almilosz left a comment (openvinotoolkit/openvino#29758) https://github.com/openvinotoolkit/openvino/issues/29758#issuecomment-2889834547

This task is open now. To take this issue leave text: .take as the comment in this issue. Please, make sure that the issue has the status Contributors Needed and doesn't have another user in Assingnees.

— Reply to this email directly, view it on GitHub https://github.com/openvinotoolkit/openvino/issues/29758#issuecomment-2889834547, or unsubscribe https://github.com/notifications/unsubscribe-auth/BD2ZRIGJMQOHEJ7T5VLUNL327F6EPAVCNFSM6AAAAABZ2SB3P6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQOBZHAZTINJUG4 . You are receiving this because you were mentioned.Message ID: @.***>

somilshivhare avatar May 19 '25 07:05 somilshivhare

Hello @somilshivhare! You can start from looking at the last two PRs to this issue. You can find there a lot of comments from me and @Retribution98.

almilosz avatar May 19 '25 09:05 almilosz

hello @almilosz,

I've created a new pull request implementing the Tensor.copyTo() method in the JavaScript API. This PR includes:

  • Implementation of the copyTo() method in the Node.js addon
  • Added corresponding TypeScript definitions
  • Included test cases for the new functionality

The changes span across 3 files with 110 new lines of code, focusing on the JavaScript bindings for tensor operations.

Please review when you have time. Let me know if you need any clarification or have suggestions for improvements.

Thanks!

somilshivhare avatar May 20 '25 04:05 somilshivhare

could you linked it?

mlukasze avatar May 20 '25 04:05 mlukasze

sure here it is https://github.com/openvinotoolkit/openvino/pull/30623#issue-3074283375

somilshivhare avatar May 20 '25 05:05 somilshivhare

.take

dewanganmehul avatar Jun 15 '25 14:06 dewanganmehul

Hello @dewanganmehul How is it going? Did you have any troubles with these steps?

almilosz avatar Jun 24 '25 09:06 almilosz

Hi! I would like to work on this issue for my open-source college project. Could you please assign it to me?

Thanks!

jidnyasa18 avatar Aug 06 '25 17:08 jidnyasa18

.take

SrishtikSekar avatar Sep 09 '25 18:09 SrishtikSekar

Hello @jidnyasa18 If you are still interested you can take this one: https://github.com/openvinotoolkit/openvino/issues/29757

almilosz avatar Sep 10 '25 14:09 almilosz

@almilosz if there is no progress i would like to work on it .Also it would be very helpfull if you review my PR for #32061

Sujanian1304 avatar Sep 15 '25 18:09 Sujanian1304

@SrishtikSekar are you still working on the PR?

almilosz avatar Sep 16 '25 12:09 almilosz

hey @almilosz if the issue is still open i am happy to work on it

Sujanian1304 avatar Oct 07 '25 14:10 Sujanian1304

Hello, I assigned you. Good luck!

almilosz avatar Oct 08 '25 08:10 almilosz

Hi, I’d like to work on this issue as part of the GSoC prerequisite task. I’ve read the contribution guide and set up my environment. Could you please assign this issue to me? Thank you!

Yashwanth1124 avatar Nov 02 '25 15:11 Yashwanth1124