Skip to content

Conversation

@naomi-simumba
Copy link
Collaborator

provide option to do tiled inference during validation

Signed-off-by: Naomi Simumba <[email protected]>
@naomi-simumba naomi-simumba self-assigned this Jun 17, 2025
@naomi-simumba naomi-simumba linked an issue Jun 17, 2025 that may be closed by this pull request
@naomi-simumba naomi-simumba changed the title tiled_inference on val [WiP] tiled inference on validation Jun 17, 2025
Signed-off-by: Naomi Simumba <[email protected]>
@Joao-L-S-Almeida
Copy link
Collaborator

HI, @naomi-simumba I agree with the modifications, but my only doubt is if it makes sense to use the tiled inference during the validation, since it affects the quality of the evaluation. Could it negatively impact the fine-tuning ?

Joao-L-S-Almeida and others added 2 commits June 20, 2025 14:21
@Joao-L-S-Almeida
Copy link
Collaborator

@naomi-simumba Do you want to proceed with the merging or prefer to update something before it ?

@naomi-simumba naomi-simumba changed the title [WiP] tiled inference on validation tiled inference on validation Jul 30, 2025
@naomi-simumba
Copy link
Collaborator Author

@Joao-L-S-Almeida ,
I am ready to merge now.

There is another issue when using DOFA + tiled_inference where the difference in image sizes between train and val causes an error during the sanity check. This would address that.

@Joao-L-S-Almeida Joao-L-S-Almeida added 1.1 and removed 1.2 labels Jul 30, 2025
@Joao-L-S-Almeida Joao-L-S-Almeida merged commit b069eac into main Jul 30, 2025
4 of 7 checks passed
@Joao-L-S-Almeida Joao-L-S-Almeida deleted the tiled_inference_on_val branch July 30, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tiled inference during validation

4 participants