Skip to content

fix(processor): horizontally flip LIBERO observations#3882

Open
ogarciarevett wants to merge 2 commits into
huggingface:mainfrom
ogarciarevett:codex/libero-hflip-observations
Open

fix(processor): horizontally flip LIBERO observations#3882
ogarciarevett wants to merge 2 commits into
huggingface:mainfrom
ogarciarevett:codex/libero-hflip-observations

Conversation

@ogarciarevett

Copy link
Copy Markdown

Closes #3830

The current LIBERO processor flips both image axes, which turns the observation into a 180° rotation instead of the horizontal flip discussed in the issue.

This changes the processor to flip only the width dimension and adds a regression test that checks the image output directly.

Tested:

  • uv run python -m pytest tests/processor/test_libero_processor.py -q

@github-actions github-actions Bot added tests Problems with test coverage, failures, or improvements to testing processor Issue related to processor labels Jun 27, 2026

@Liyux3 Liyux3 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I locally verified this pr, and the numpy and torch flip paths in the test produced matching results

@Liyux3 Liyux3 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good, tested and works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

processor Issue related to processor tests Problems with test coverage, failures, or improvements to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

libero image observations need hflip

2 participants