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

Added TimeDistributed option #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

voletiv
Copy link

@voletiv voletiv commented Nov 21, 2017

Often, CNN+LSTM architectures require a TimeDistributed version of CNNs - one CNN architecture acting on several time-distributed inputs, and then fed into an LSTM.

Invoking the argument time_distributed=True shall create a TimeDistributed version of resnet, with nb_time_steps versions of the resnet, the input_shape having been specified as (nb_time_steps, nb_rows, nb_cols, nb_channels).

@voletiv
Copy link
Author

voletiv commented Nov 21, 2017

All tests passed, except on Python 2.7 with Theano - because 'mkl' could not be imported:

RuntimeError:
Could not import 'mkl'. Either install mkl-service with conda or set
MKL_THREADING_LAYER=GNU in your environment for MKL 2018.
If you have MKL 2017 install and are not in a conda environment you
can set the Theano flag blas.check_openmp to False. Be warned that if
you set this flag and don't set the appropriate environment or make
sure you have the right version you will get wrong results.

@ahundt
Copy link

ahundt commented Nov 25, 2017

You might want to consider submitting this at https://github.com/farizrahman4u/keras-contrib, which is more up to date

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

Successfully merging this pull request may close these issues.

2 participants