Skip to content

Moved forward tiling patch to Model API#629

Open
A-Artemis wants to merge 2 commits into
masterfrom
aurelien/forward-tiling
Open

Moved forward tiling patch to Model API#629
A-Artemis wants to merge 2 commits into
masterfrom
aurelien/forward-tiling

Conversation

@A-Artemis

Copy link
Copy Markdown

What does this PR do?

Added predict tiles to handle tiling during evaluation step of training for detection and instance/semantic segmentation.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@A-Artemis A-Artemis self-assigned this Jul 20, 2026
@A-Artemis
A-Artemis requested a review from a team as a code owner July 20, 2026 09:38
@github-actions github-actions Bot added tests Related to tests python python related changes labels Jul 20, 2026

@tybulewicz tybulewicz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Apart from my question, please run uvx pre-commit run --all-files to fix code styling issues reported by the CI.

Comment thread model_api/src/model_api/tilers/tiler.py

@tybulewicz tybulewicz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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

Labels

python python related changes tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants