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.

Sign up for GitHub

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

Jump to bottom

Prepare v2.2.0 #435

Merged
merged 16 commits into from
Jul 3, 2023
Merged

Prepare v2.2.0 #435

merged 16 commits into from
Jul 3, 2023

Conversation

stefaniapedrazzi
Copy link
Collaborator

@stefaniapedrazzi stefaniapedrazzi commented May 16, 2023 •
edited
Loading

Update version

  • changelog
  • workflows
  • documentation
  • ros/ros2

Fix tests:

  • Test wheel: avoid version mismatch by first searching for local wheel packages
  • Docker build and test "no space left on device" error: use "free-disk-space" action
  • Docker tests:
    • perception/object_tracking_2d fails, it seems to be due to excessive memory use in the CI/Docker test (locally works fine):
      reduce batch_size to 2
    • control/multi_object_search fails because it requires CUDA: test is disabled

@stefaniapedrazzi stefaniapedrazzi self-assigned this May 16, 2023
@stefaniapedrazzi stefaniapedrazzi added the test release Tests if a wheel created from a branch runs correctly label Jun 22, 2023
@stefaniapedrazzi stefaniapedrazzi added the test sources Run style checks label Jun 28, 2023
@stefaniapedrazzi
Copy link
Collaborator Author

Finally all the enabled tests are passing:
https://github.com/opendr-eu/opendr/actions/runs/5401784419

They have now been restarted when adding the test sources label needed to trigger the tests that otherwise will block the merge.

@stefaniapedrazzi stefaniapedrazzi marked this pull request as ready for review June 28, 2023 15:15
@stefaniapedrazzi
Copy link
Collaborator Author

This PR is now ready for review.
Then when merged, I will release the new version of the toolkit.

passalis
passalis approved these changes Jul 3, 2023
View reviewed changes
Copy link
Collaborator

@passalis passalis left a comment

Choose a reason for hiding this comment

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

Thank you!

omichel
omichel approved these changes Jul 3, 2023
View reviewed changes
Copy link
Collaborator

@omichel omichel left a comment

Choose a reason for hiding this comment

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

Thank you.

@stefaniapedrazzi stefaniapedrazzi merged commit ca76ab9 into master Jul 3, 2023
@stefaniapedrazzi stefaniapedrazzi deleted the version-bump branch July 3, 2023 12:54
lucamarchionni pushed a commit to lucamarchionni/opendr that referenced this pull request Jun 10, 2024
@stefaniapedrazzi
Prepare v2.2.0 (opendr-eu#435)
881fe7b 
* Version bump

* Update version in ros/ros2 packages

* Remove duplicated line

* Update workflows

* Try to free some disk space

* Update tests_suite.yml

* Update tests_suite.yml

* Update tests_suite_develop.yml

* Fix deprecation errors in tests_suite.yml

* Disable docker test of controller/multi_object_search

* Update actions version

* Debug by skipping deleting the docker artifact

* Debug perception/object_tracking_2d

* Restore all tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@passalis passalis passalis approved these changes

@omichel omichel omichel approved these changes

Assignees

@stefaniapedrazzi stefaniapedrazzi

Labels
test release Tests if a wheel created from a branch runs correctly test sources Run style checks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stefaniapedrazzi @omichel @passalis