piergiaj / super-events-cvpr18

Code for our CVPR 2018 paper "Learning Latent Super-Events to Detect Multiple Activities in Videos"
MIT License
123 stars 34 forks source link

centers always multiply by zeros since variable always equal to zeros #11

Open Lilyo opened 3 years ago

Lilyo commented 3 years ago

Thanks for your great work! but I found a problem below...

in your code _super-events-cvpr18/temporal_structurefilter.py line 41-46

def get_filters(self, delta, gamma, center, length, time):
      ...
        a = Variable(torch.zeros(self.Ni))
        a = a.cuda()

        # stride and center
        a = deltas[:, None] * a[None, :]
        a = centers[:, None] + a

why always multiply deltas to zeros? do you need to exchange line 45 and line 46