fel88 / Dendrite

ONNX viewer/editor/inference
MIT License
12 stars 2 forks source link

Provide Dagre unit tests and onnx model used #1

Open GeorgeS2019 opened 2 years ago

GeorgeS2019 commented 2 years ago

I tried using squeezenet.onnx

The Dagre layout crashes!.

Need unit test project to sort out the Dagre layout. Keep it up!

Table layout of squeezenet.onnx

ItKwJv70CZ

GeorgeS2019 commented 2 years ago

There is a .NET graphlayout methods used here to layout deep learning model. Perhaps this could be incorporated into your project?

Seq2Seq transformer model layout

image

fel88 commented 2 years ago

Dagre layout is not finished yet. I'm working on it right now. Hope it will be ready soon (maybe in 3-7 days)

GeorgeS2019 commented 2 years ago

@fel88 U are attempting to port a Javascript Dagre graph layout engine for .NET community. This is not trivial and thank you for your contribution. Also thank you for your vision that the .NET community needs an ONNX viewer and editor. Great job!!!

FYI: We work hard to persuade the TorchSharp team to make it easy for .NET community to port python pytorch code to c#. Likewise, I urge you to consider that (eventually) in your Dagre implementation (make it a reusable standalone library). This will get more .NET users to use your Dagre implementation and speed up the quality of the implementation.

Reference https://github.com/rwl/dagre ( Dagre for Dart) => FYI. We need Dagre for .NET :-)

fel88 commented 2 years ago

I haven't finished porting completely yet, but your model (squeezenet1.0-9) should work now.

GeorgeS2019 commented 2 years ago

@fel88 Impressive

RhgCUpws0H

GeorgeS2019 commented 2 years ago

Update on GPT2.onnx model

and the discussion associated with it