search

patlevin / tfjs-to-tf

A TensorFlow.js Graph Model Converter
MIT License
139 stars 18 forks source link

closes #39 #40

Closed patlevin closed 2 years ago

patlevin commented 2 years ago

Fix Issue #39

Root cause: duplicate node names during fused op splitting. Fixed by simplifying node name generation.

Changes

  1. Replaced overcomplicated node name generation - unique names are guaranteed by simply appending split node op names to the original node name
  2. Added missing model file to unit tests model generator (unit tests didn't pass otherwise - only noticed this since I changed dev machines)
  3. Fixed some test data (test models were a little off)
  4. Switched build specs to PEP 518 and PEP 621
  5. No-Op nodes with dummy references caused issues and are ignored during conversion (i.e. copied as-is)