Skip to content

Add pytests#127

Merged
aaravpandya merged 29 commits intomainfrom
ap_pytests
May 23, 2024
Merged

Add pytests#127
aaravpandya merged 29 commits intomainfrom
ap_pytests

Conversation

@aaravpandya
Copy link
Copy Markdown
Collaborator

@aaravpandya aaravpandya commented May 20, 2024

This PR adds pytests which does the following

  1. Adds support for python based tests. Specifically we are adding a test that runs through expert trajectories and ensure all agents reach goal and do not collide.
  2. Adds support for calling ctest directly. So we dont need to call cpp tests seperately. Running pytest is enough.

This PR is based on #130

@aaravpandya aaravpandya marked this pull request as ready for review May 22, 2024 01:50
@aaravpandya aaravpandya merged commit dee6972 into main May 23, 2024
@aaravpandya aaravpandya deleted the ap_pytests branch October 4, 2024 03:20
wangbingke0 pushed a commit to wangbingke0/gpudrive that referenced this pull request Feb 13, 2026
* Add pytests

* test experts

* Changed the padding position (Emerge-Lab#126)

* Mark agents as static if they spawn near goal (Emerge-Lab#125)

* Init static agents

* Fix the compile error

* Formatting

* Add some rendering support

* Check the first valid state against goal

* Working test on experts

* Configure pytests

* Add pytest to github workflow

* remove sample test

* Updated poetry lock

* Remove unnecassry imports

* Build in release mode

* Remove poetry lock file

* Reduced consts to use less memory

* Update manual.yml

* upload dumps

* Dont init agents if not valid on first timestep

* Add param

* Change name and fix test

* Change comment

* Modified workflow

* cache workflow downloads

* Revert prev changes
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

Successfully merging this pull request may close these issues.

2 participants