microsoft / VideoX

VideoX: a collection of video cross-modal models
Other
967 stars 160 forks source link

confusion of **drop path** #85

Closed ignitemylife closed 1 year ago

ignitemylife commented 1 year ago

https://github.com/microsoft/VideoX/blob/master/X-CLIP/models/cct.py#L52

we see drop path usually at BatchSize dimension, but in this repo., I find it is applied at token length dimension, is there some mistakes or misconception here?

nbl97 commented 1 year ago

It is a mistake, and we will fix that later in case of misleading. Thanks for your interest and reminder.

ignitemylife commented 1 year ago

thank you for your reply! Best wishes