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

Why you put the GRU in the loop of n_layers? #6

Open
wuhaoran111 opened this issue Apr 20, 2021 · 2 comments
Open

Why you put the GRU in the loop of n_layers? #6

wuhaoran111 opened this issue Apr 20, 2021 · 2 comments

Comments

@wuhaoran111
Copy link

wuhaoran111 commented Apr 20, 2021

Dear author,
i have read your code like model_graph.py and i found that the GRUs for the pedestrian and context are in the loop of the n_layers of the GNN, the structure is like follows:

    for batch_size:
        for time_step:
            for gnn_layer:
                 curr_ped, h_ped = self.ped_gru(curr_ped, h_ped) # the GRU for the pedestrian
                 ......

In this structure the GRU updates time_step*gnn_layer steps. But in my perspective, to simulate the temporal relation, the GRU should only update time_step steps. And this makes me confused, could you give me an explanation?

@MrYu001
Copy link

MrYu001 commented Jan 21, 2022

Hello bro, have you run the code on the JAAD dataset, i think the JAAD dataset needs to be processed and saved as .npy file, but the author of this repository did not provide it.

@wuhaoran111
Copy link
Author

wuhaoran111 commented Jan 21, 2022 via email

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