Skip to content

Refine Crop transform documentation and update installation instructions#103

Closed
GabrielBG0 wants to merge 9 commits into
discovery-unicamp:mainfrom
GabrielBG0:main
Closed

Refine Crop transform documentation and update installation instructions#103
GabrielBG0 wants to merge 9 commits into
discovery-unicamp:mainfrom
GabrielBG0:main

Conversation

@GabrielBG0
Copy link
Copy Markdown
Collaborator

@GabrielBG0 GabrielBG0 commented Apr 12, 2026

Describe your changes

  • Fix outdated pip install minerva-ml references and update Crop transform tests
  • Updated installation instructions to use the correct package name (minerva) and fixed the Crop transform tests to match the CHW format introduced in commit 8d12499.
  • Add auto format CI step

Changes:

  • README.md and docs/installation.md: replaced all pip install minerva-ml references with pip install minerva, and updated PyPI links from /project/minerva-ml/ to /project/minerva/.
  • tests/transforms/test_transform.py: updated test_crop_output_shape and test_crop_with_padding to use CHW-format arrays (C, H, W) instead of HWC (H, W, C), matching the updated Crop implementation. Added test_crop_2d_input and test_crop_bbox to cover the 2D path and the bbox parameter, both of which were also changed in the same refactor.
  • Add auto-format.yml to auto format code on PR.

Issue ticket number and link (If apply)

N/A

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added tests to this functionality.
  • All tests are passing.
  • I have updated the documentation as needed.
  • I have followed the project's coding guidelines.

@GabrielBG0 GabrielBG0 requested a review from otavioon April 12, 2026 15:53
@GabrielBG0 GabrielBG0 closed this Apr 16, 2026
@GabrielBG0 GabrielBG0 removed the request for review from otavioon April 16, 2026 20:55
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.

1 participant