Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Question]: Loading Keras models into appropriate TSP layers automatically #219

Closed
Marviel opened this issue Mar 11, 2019 · 2 comments
Closed
Assignees

Comments

@Marviel
Copy link

Marviel commented Mar 11, 2019

Hey, awesome project! Forgive me if this question is answered elsewhere.

I noticed that the InceptionV3 example seems to manually specify the javascript definition of the Tensorspace models, even though I think that the same information could be inferred given the .json file within that example.

Am I wrong about this? I know there is an issue with some backends (tf.js?) not providing enough information for automated extraction and encoding into Tensorspace models, but was under the impression that Keras based models contained the necessary data.

Have I messed up in my understanding of the .json keras model definitions?

@Marviel
Copy link
Author

Marviel commented Mar 11, 2019

Oh, this may just be solved by #213, when it closes :)

@syt123450 syt123450 self-assigned this Mar 12, 2019
@syt123450
Copy link
Member

@Marviel Thanks for your suggestions!

As you said, tfjs model can provide some information for TensorSpace layers, and it is possible to get these information automatically, and I am designing the auto-loader to automatically create TensorSpace model with given pre-trained model.

Theoretically, it is possible to get all information from tfjs layer model or graph model. As TensorSpace mainly used to visualize a pre-trained model, some layers will not take effect in model prediction process, such as, batchnorm, dropout, and so on, and these layers will not be visualized in TensorSpace. From my perspective, auto-loader needs to infer a sub-set of pre-trained model's topo structure, it is a little bit complex, and may take time to design and implement it.

I am actively developing this feature, it will come in future version of TensorSpace~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants