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

Adds test for camera to check that different image sizes work #964

Merged
merged 12 commits into from
Sep 19, 2024

Conversation

pascal-roth
Copy link
Collaborator

@pascal-roth pascal-roth commented Sep 9, 2024

Description

Adds test to check that different image sizes are respected by the camera implementation.

Fixes #165

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have run the pre-commit checks with ./isaaclab.sh --format
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the changelog and the corresponding version in the extension's config/extension.toml file
  • I have added my name to the CONTRIBUTORS.md or my name already exists there

@pascal-roth pascal-roth self-assigned this Sep 9, 2024
@pascal-roth pascal-roth added the enhancement New feature or request label Sep 9, 2024
Copy link
Contributor

@Mayankm96 Mayankm96 left a comment

Choose a reason for hiding this comment

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

NIT: Can we split the test_multi_camera into two different tests? Just to make it clearer to catch the issues in the future? Otherwise, looks fine. Thanks!

pascal-roth and others added 4 commits September 11, 2024 14:49
Co-authored-by: Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Signed-off-by: Pascal Roth <57946385+pascal-roth@users.noreply.github.com>
@pascal-roth
Copy link
Collaborator Author

@Mayankm96 I split up the test, should be ready to be merged now

Signed-off-by: Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Copy link
Contributor

@Mayankm96 Mayankm96 left a comment

Choose a reason for hiding this comment

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

@kellyguo11 we should add a similar test for the tiled camera too. Tagging you for vis :)

@Mayankm96 Mayankm96 changed the title Adds test to check that different image sizes are respected Adds test for camera to check that different image sizes work Sep 12, 2024
Copy link
Collaborator

@jsmith-bdai jsmith-bdai left a comment

Choose a reason for hiding this comment

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

LGTM!

pascal-roth and others added 2 commits September 18, 2024 09:23
Co-authored-by: James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Signed-off-by: Pascal Roth <57946385+pascal-roth@users.noreply.github.com>
@Mayankm96 Mayankm96 merged commit 55ab947 into main Sep 19, 2024
2 of 3 checks passed
@Mayankm96 Mayankm96 deleted the test/diff-img-sizes branch September 19, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug Report] Camera output resolution stays the same across camera instances
3 participants