Skip to content

Commit 8acb618

Browse files
committed
bug fixed
1 parent 9d2f2b7 commit 8acb618

8 files changed

Lines changed: 18 additions & 19 deletions

File tree

.github/pull_request_template.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ Relates to #
5050

5151
<!-- Describe the tests you ran to verify your changes -->
5252

53-
- [ ] Ran scripts/quick_test.sh successfully
54-
- [ ] Ran scripts/multigpu_quick_test.sh successfully (if applicable)
53+
- [ ] Ran scripts/closed_loop_test.sh successfully
54+
- [ ] Ran scripts/multigpu_closed_loop_test.sh successfully (if applicable)
5555
- [ ] Added new unit tests
5656
- [ ] All existing tests pass
5757
- [ ] Manual testing performed

.github/workflows/tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,8 @@ jobs:
6767
echo " - Significant compute resources"
6868
echo ""
6969
echo "These tests should be run manually using:"
70-
echo " bash scripts/quick_test.sh"
71-
echo " bash scripts/multigpu_quick_test.sh"
70+
echo " bash scripts/closed_loop_test.sh"
71+
echo " bash scripts/multigpu_closed_loop_test.sh"
7272
echo ""
7373
echo "Consider setting up self-hosted runners for automated GPU testing."
7474

CONTRIBUTING.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ By participating in this project, you agree to abide by our [Code of Conduct](CO
5656
5. **Verify installation**:
5757
```bash
5858
# Run quick test
59-
bash scripts/quick_test.sh
59+
bash scripts/closed_loop_test.sh
6060
```
6161

6262
## Development Workflow
@@ -168,10 +168,10 @@ Before submitting a pull request, ensure all tests pass:
168168

169169
```bash
170170
# Run quick test
171-
bash scripts/quick_test.sh
171+
bash scripts/closed_loop_test.sh
172172

173173
# Run distributed test (multi-GPU)
174-
bash scripts/multigpu_quick_test.sh
174+
bash scripts/multigpu_closed_loop_test.sh
175175
```
176176

177177
### Writing Tests

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,10 +166,10 @@ Verify your installation with a pre-trained model:
166166
export WORLDENGINE_ROOT=$(pwd)
167167

168168
# Option 1: Single GPU test
169-
bash scripts/quick_test.sh
169+
bash scripts/closed_loop_test.sh
170170

171171
# Option 2: Multi-GPU test (Default 8 GPUs)
172-
bash scripts/multigpu_quick_test.sh
172+
bash scripts/multigpu_closed_loop_test.sh
173173
```
174174

175175
**What this does:**

docs/quick_start.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ cd /path/to/WorldEngine
3535
export WORLDENGINE_ROOT=$(pwd)
3636

3737
# Run quick test
38-
bash scripts/quick_test.sh
38+
bash scripts/closed_loop_test.sh
3939
```
4040

4141
**Expected output:**
@@ -60,7 +60,7 @@ cd /path/to/WorldEngine
6060
export WORLDENGINE_ROOT=$(pwd)
6161

6262
# Run multi-GPU quick test (8 splits in parallel)
63-
bash scripts/multigpu_quick_test.sh
63+
bash scripts/multigpu_closed_loop_test.sh
6464
```
6565

6666
**Expected output:**
@@ -77,7 +77,7 @@ Results merged to: experiments/closed_loop_exps/e2e_vadv2_50pct/navtest_failures
7777

7878
**Time:** ~2-3 minutes with 8 GPUs
7979

80-
**Note:** If you have fewer than 8 GPUs, edit `scripts/multigpu_quick_test.sh` and change the loop `{0..7}` to match your GPU count (e.g., `{0..3}` for 4 GPUs).
80+
**Note:** If you have fewer than 8 GPUs, edit `scripts/multigpu_closed_loop_test.sh` and change the loop `{0..7}` to match your GPU count (e.g., `{0..3}` for 4 GPUs).
8181

8282
---
8383

@@ -129,7 +129,7 @@ For more details on the testing pipeline, see:
129129

130130
### Change Test Scenarios
131131

132-
Edit `scripts/quick_test.sh` to test on different scenarios:
132+
Edit `scripts/closed_loop_test.sh` to test on different scenarios:
133133

134134
```bash
135135
# Original (navtest rare cases num: 288)
@@ -147,7 +147,7 @@ bash scripts/run_testing.sh \
147147

148148
### Change Model Checkpoint
149149

150-
Edit the checkpoint path in `scripts/quick_test.sh`:
150+
Edit the checkpoint path in `scripts/closed_loop_test.sh`:
151151

152152
```bash
153153
bash scripts/run_testing.sh \

projects/AlgEngine/mmdet3d_plugin/datasets/navsim_openscene_nuplan.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -672,12 +672,12 @@ def update_canbus(self, input_dict, index):
672672
sdc_loc_lidar = input_dict["e2l"] @ sdc_loc_ego
673673

674674
# Rotation
675-
if "ego2global" in info:
675+
if "ego2global_rotation" in info:
676+
e2g_r = info["ego2global_rotation"]
677+
sdc_rot_global_quat = Quaternion(e2g_r)
678+
else:
676679
e2g_r_mat = info["ego2global"][:3, :3]
677680
sdc_rot_global_quat = Quaternion(matrix=e2g_r_mat)
678-
else:
679-
e2g_r = info.get("ego2global_rotation", [1, 0, 0, 0])
680-
sdc_rot_global_quat = Quaternion(e2g_r)
681681

682682
sdc_rot_global_ele = sdc_rot_global_quat.elements
683683
yaw_angle_global = quaternion_yaw(sdc_rot_global_quat) / np.pi * 180
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
# export WORLDENGINE_ROOT="/path/to/your/WorldEngine"
22

3-
# Default 8 GPUs, you can change it to 4 or 2 if you have fewer GPUs
43
bash projects/SimEngine/scripts/run_ray_distributed_testing.sh \
54
$WORLDENGINE_ROOT/projects/AlgEngine/configs/worldengine/e2e_vadv2_50pct.py \
65
$WORLDENGINE_ROOT/data/alg_engine/ckpts/e2e_vadv2_50pct_ep8.pth \

0 commit comments

Comments
 (0)