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

Run long video #53

Open
shuyansy opened this issue Apr 26, 2024 · 8 comments
Open

Run long video #53

shuyansy opened this issue Apr 26, 2024 · 8 comments

Comments

@shuyansy
Copy link

Thanks for your work. I want to test my long video and set the middle_video to 0. However, i met this error. Could you help me to provide some solutions? Thanks so much!

截屏2024-04-26 下午9 24 11

@Espere-1119-Song
Copy link
Collaborator

I think I have fixed the bug before. Can u tell me which code u use? The error often occurs when u didn't initialize the long-term memory when update a new long video.

@shuyansy
Copy link
Author

Hi, I just run the main branch. Moreover, i found "# middle_video = middle_video == 1" In line 368 in inference.py, So I delete it Do you have any ideas. Thanks so much for your reply!

@Espere-1119-Song
Copy link
Collaborator

image
I think line 368 is not commented out.

@shuyansy
Copy link
Author

Thanks so much! The code can run normally. However, I want to read more frames so I adjust " N_SAMPLES = 128", when I use larger number, the error still occurs. Do you have any ideas? Thanks for your time and response!

@Espere-1119-Song
Copy link
Collaborator

I think you need to check the actual length of positional embedding.

@shuyansy
Copy link
Author

Thanks for the response. I expand "n_position = 16" to" n_position = 32" so it can support more frames. However I found the max value is 32 due to the size of the pretrained model. Thus, the max frame to model is about 500. Is my understanding right?

@Espere-1119-Song
Copy link
Collaborator

I think you can explore it by checking the length of long-term memory :). The max frame read by the model is len(short-term memory) * len(long-term memory) / 2

@shuyansy
Copy link
Author

Thanks for reminder! The last question I want to check is whether my hyperparameter setting is right: cur_sec=1 cur_min=1 If I want to read the whole long video.

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

No branches or pull requests

2 participants