+
Skip to content

Conversation

vmoens
Copy link
Collaborator

@vmoens vmoens commented Sep 11, 2025

No description provided.

Copy link

pytorch-bot bot commented Sep 11, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/rl/3165

Note: Links to docs will display an error until the docs builds have been completed.

❌ 3 New Failures, 23 Pending, 4 Unrelated Failures

As of commit c9f906c with merge base 75ca4b4 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 11, 2025
@vmoens vmoens force-pushed the fix-torch.2.1 branch 7 times, most recently from 7f27660 to 7ff405c Compare September 11, 2025 10:12
Comment on lines +2785 to +2790
# Skip football scenario due to VMAS bug: IndexError in get_wall_separations
if scenario_name == "football":
pytest.skip(
"Football scenario has a shape mismatch bug in VMAS get_wall_separations method"
)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matteobettini looks like football is broken - should be easy to reprod

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the heads up man!

I should have fixed it proroklab/VectorizedMultiAgentSimulator@7eff87c

I'll make a PR when this is on pypi

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful thanks!

@vmoens vmoens added Tests Incomplete or broken unit tests CI Has to do with CI setup (e.g. wheels & builds, tests...) labels Sep 11, 2025
@vmoens vmoens merged commit 66b9a21 into main Sep 11, 2025
50 of 71 checks passed
@vmoens vmoens deleted the fix-torch.2.1 branch September 11, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Has to do with CI setup (e.g. wheels & builds, tests...) CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Tests Incomplete or broken unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载