epic-kitchens/C1-Action-Recognition-TSN-TRN-TSM

Minor inconsistency in the gulp adaptor.

Closed this issue · 1 comments

Hi, I found that in the RGB gulp adaptor, the stop_frame is inclusive.

for idx in range(meta["start_frame"], meta["stop_frame"] + 1)

whereas in the flow adaptor, the stop_frame is excluded.

for idx in range(start_frame, stop_frame)

I know that it makes little difference, but just wanted to point it out because I found it when I was adapting the code to my training pipeline and I had an error trying to read one more frame.

Thanks for noting that :-)