Skip to content

Conversation

@SujoyKG
Copy link

@SujoyKG SujoyKG commented Jan 10, 2022

New PR

Video Classification Model Examination
Video Classification Model Examination
Copy link
Contributor

@fchollet fchollet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

Is there a realistic video classification dataset (e.g. action classification) that you could use? I don't think this model architecture would perform well on a real dataset. In addition, the architecture used seems fundamentally expensive (e.g. use of conv2d instead of separable conv, use of LSTM) which contradicts the claim that it is optimized for CPU. At equal parameter count, much leaner architectures are possible.

I would recommend reworking this into a more conventional video classification example. Please start by finding an appropriate dataset, then we can discuss what architecture we should use and whether to use a pretrained base model.

@SujoyKG
Copy link
Author

SujoyKG commented Jan 12, 2022

Thanks for the PR!

Is there a realistic video classification dataset (e.g. action classification) that you could use? I don't think this model architecture would perform well on a real dataset. In addition, the architecture used seems fundamentally expensive (e.g. use of conv2d instead of separable conv, use of LSTM) which contradicts the claim that it is optimized for CPU. At equal parameter count, much leaner architectures are possible.

I would recommend reworking this into a more conventional video classification example. Please start by finding an appropriate dataset, then we can discuss what architecture we should use and whether to use a pretrained base model.

Thanks for the reply. Please note what I am trying to express through this example. The dataset generated is optimized for quick checking of any Video Classification Model; the model is not optimized for CPU, as you have asked. I am more focused to the synthetic dataset generation rather than model building. Before feeding the actual data (real data) to the model, this generated dataset could get feeded to check the model quickly at high level (& without any GPU support).

@SujoyKG
Copy link
Author

SujoyKG commented Apr 29, 2022

Would you like to get it added to "Quick Keras Recipes"?

@pcoet
Copy link
Contributor

pcoet commented Aug 15, 2023

Thanks for the PR, @SujoyKG. Are you planning to implement this feedback?

I would recommend reworking this into a more conventional video classification example. Please start by finding an appropriate dataset, then we can discuss what architecture we should use and whether to use a pretrained base model.

Let us know if you're still working on this. Otherwise we'll close the request. Thanks!

@SujoyKG
Copy link
Author

SujoyKG commented Aug 15, 2023 via email

@github-actions
Copy link

This PR is stale because it has been open for 14 days with no activity. It will be closed if no further activity occurs. Thank you.

@github-actions github-actions bot added the stale label Aug 30, 2023
@github-actions
Copy link

This PR was closed because it has been inactive for 28 days. Please reopen if you'd like to work on this further.

@github-actions github-actions bot closed this Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants