[Good First Issue]: Enable Model.getOps() method

[almilosz]

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::Model::get_ops() method.
From JavaScript side it will be Model.getOps() method.
This method:

• Doesn't have parameters
• Returns array of ov::Node objects
  Node.js API doesn't have bindings for ov::Node. You need implement bindings
  for it. ov::Node has a lot of methods, you shouldn't implement all of them.
  Implement:
  • ov::Node::get_name()
    This method don't have parameters, it should be easy to bind it.

Useful links

• C++ API implementation of Model::get_ops() method
• Node.js API has similar implementation for Output<ov::Node>, use this as the
  reference to implement ov::Node class.

What needs to be done?

• Create NodeWrap class that inherits from Napi::ObjectWrap and represents ov::Node, create a separate header and source file.
• Implement NodeWrap as minimal wrapper of ov::Node with ov::Node.get_name() method.
• Add ModelWrap.get_ops() method to ./src/bindings/js/node/src/model_wrap.cpp
• Update TypeScript definitions in ./src/bindings/js/node/lib/addon.ts
• Create unit test for added functionality using Node.js Test Runner. It might look similar to this (Python code as an example):

  model_operators = [op.get_name().split("_")[0] for op in model.get_ops()]
     expected_ops = [
         "Subtract",
         "Transpose",
     ]
     assert len(model_operators) == 14
     for op in expected_ops:
         assert op in model_operators

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

• [OV JS] Expose Model.setFriendlyName and Model.getFriendlyName methods to Node.js api #23743
• [JS API] Add config param to Core.import_model() #24023

You may use this unfinished PR as the reference

• Enabled Model.getOps() method #25470

Resources

• [NEW] Contribution Guide
• JavaScript API Developer Documentation
• Node.js API Reference
• Node.js bindings examples of usage
• What is OpenVINO?
• How to contributing to OpenVINO
• User documentation

Contact points

@almilosz @vishniakov-nikolai

Please, mention us in this issue discussion if you have any questions.

[spran180]

.take

[github-actions]

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

[p-wysocki]

Reopening the task as the PR seems to have been abandoned.

[nashez]

@p-wysocki @almilosz Can I take this up if no one is working on it?

[mlukasze]

it would be great, have fun!

[qxprakash]

.take

[github-actions]

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

[qxprakash]

@almilosz I will give it a shot

[qxprakash]

@almilosz @vishniakov-nikolai sorry I got a but busy I'll raise a pr soon

[almilosz]

If you have any issues with it, don't hesitate to ask :)

[qxprakash]

@almilosz can you please assign this to someone else , I have been busy a lot lately so couldn't make much progress , it's better to let someone else have a shot.

[mlukasze]

no worries @qxprakash
thank you for your time, hope to see you again, later :)

task has been moved back to the available ones

[itsaflamingo]

@almilosz I'd like to take up this issue:)

[mlukasze]

it's yours, @itsaflamingo
have fun