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

Do not list enocoders in [main] configuration #27

Closed
jlibovicky opened this issue Jun 27, 2016 · 5 comments
Closed

Do not list enocoders in [main] configuration #27

jlibovicky opened this issue Jun 27, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@jlibovicky
Copy link
Contributor

Now, the encoders are listed multiple times in the configuration: in the main configuration and as the arguments of a decoder. Duplication is a frequent source of errors, and therefore it should be only in the decoder.

@jlibovicky jlibovicky added this to the pre-MTM milestone Jun 27, 2016
@jlibovicky jlibovicky self-assigned this Jun 27, 2016
@jindrahelcl jindrahelcl reopened this Jul 11, 2016
@jindrahelcl
Copy link
Member

Problém teďkon je v tom, že training loop potřebuje všechny *kodéry, aby vytvořil feed_dicty. Co kdybychom převedly zodpovědnost za vytváření enkodérských feed_dictů na decoder? Vlastně by to kopírovalo strom, ve kterým jsou ty objekty pospojovaný v konfiguraci. Každej objekt by měl zodpovědnost za vytváření feed_dictů svých synů (nebo otců jestli koukáš odspoda nebo odshora)

@tomasmcz
Copy link
Member

Ten training loop je vůbec problematická entita, nebylo by lepší s ním nejdřív něco udělat, než se pustíme tady do toho?

@jindrahelcl
Copy link
Member

Právě na tom dělám, to bylo zdrojem toho nápadu, kterej jsem sem připsal.

@jlibovicky
Copy link
Contributor Author

Já bych to nepřeváděl na dekodér (ale je to spíš otázka vkusu). Důvod je, že můžeme mít víc dekodérů, který budou sdílet některý z enkodérů a asi bysme se chtěli vyhnout tomu, aby se tam feedovalo dvakrát. Potom by se muselo kontrolovat, jestli už není nafeedováno. Přišlo by mi hezčí všechno, co je "feedable" dát do setu a pak to nakrmit.

@jlibovicky
Copy link
Contributor Author

Vyřešeno s #151.

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

No branches or pull requests

3 participants