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

Code for AEMCARL #7

Open
Sahil177 opened this issue Mar 6, 2023 · 6 comments
Open

Code for AEMCARL #7

Sahil177 opened this issue Mar 6, 2023 · 6 comments

Comments

@Sahil177
Copy link

Sahil177 commented Mar 6, 2023

Dear owners,

As far as I can tell it doesnt seem like the code for your proposed AEMCARL solution is included in this repo. Would it be possible to add this?

Many thanks,

Sahil

@SJWang2015
Copy link
Owner

The location of AEMCARL code at the line 90 of "AEMCARL/crowd_nav/common/qnetwork_factory.py "

@Sahil177
Copy link
Author

Sahil177 commented Mar 7, 2023

Ah I see thanks, so the train command in the get started section does in fact train your AEMCARL network since the test policy flag is set to 5?

@Sahil177
Copy link
Author

Sahil177 commented Mar 7, 2023

Another question, what test case/sim setup did you use to produce the video on the read me of the repo with many agents in crowd_sim? Also how did you get the extra graphic that shows how many GRU units are in use etc. Thanks!

@SJWang2015
Copy link
Owner

The command "python test.py --policy actenvcarl --test_policy_flag 5 --multi_process self_attention --agent_timestep 0.4 --human_timestep 0.5 --reward_increment 2.0 --position_variance 2.0 --direction_variance 2.0 --model_dir data/output --phase test --visualize --test_case 0", is used to configure the test case. The visualization of the GRU code at the line 777 of "https://github.com/SJWang2015/AEMCARL/blob/main/crowd_sim/envs/crowd_sim_video.py"

@Sahil177
Copy link
Author

Sahil177 commented Mar 8, 2023

Thanks, this test command only produces 5 agents, the sim has 17 i think and the motion patterns are different to test case 0. I tried the same command with --human_num 17 added but the motion pattern is different to the one shown in the video.

@SJWang2015
Copy link
Owner

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