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

Your Glow or RealNVP's implementation is forgot Split Layer, I think #99

Open
MokkeMeguru opened this issue Dec 18, 2019 · 3 comments
Open

Comments

@MokkeMeguru
Copy link

こんにちは、TFP から pixyz のコード翻訳を勉強しています。

現在 TFP は flow-base モデルの対数尤度をきちんと計算できていないので、こちらに全体のコードを移したいと考えています。

example の ipynb を読んでいると、Glow, RealNVP の ipynb に Split レイヤーが欠けていることに気が付きました。

本来これらの実装は、https://github.com/MokkeMeguru/glow-realnvp-tutorial/blob/master/examples/jupyter_notebooks/Glow_mnist.ipynb にあるように Split レイヤー(私の実装で言う Blockwise3D)が存在します。

現状 Splitレイヤー の実装はできますか?

@kenoharada
Copy link
Collaborator

ご指摘ありがとうございます。

Split Layer(Blockwise3D)の行なっている処理をPyTorchで実装することが可能であればSplit Layerの実装はPixyzでの実装も可能です。

Split Layer(Blockwise3D)でどのような処理が行われているか現在こちらでも確認しようと思いますが, 取り急ぎの回答とさせていただきます。

また,Pixyzへのコード移植のご検討ありがとうございます。
Pixyzについて,まだ開発中でありますがTutorialがございます,こちらもご覧になっていただけますと幸いです。
https://github.com/masa-su/pixyz/tree/feature/tutorial/tutorial

@masa-su
Copy link
Owner

masa-su commented Dec 18, 2019

@MokkeMeguru
補足します.
現状のPixyzのflowの実装は,まだマルチスケールアーキテクチャを想定しておらず,そのためinvertibleなSplitLayerについてもまだ実装しておりません.
他の部分の開発を進めていたため,後回しにしておりましたが,近日中に対応します.

Flow関係のAPIはこちらのdocsも参考にしてください.
http://docs.pixyz.io/en/latest/flows.html

@MokkeMeguru
Copy link
Author

API docs を読ませていただきました。
大変使いやすい形になっていると思います。

マルチスケールアーキテクチャはシングルスケールに比べて精度が向上するようなので1、対応を楽しみにしています。

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

No branches or pull requests

3 participants