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

Error about varying image sizes in fully supervised model training #188

Closed
hummuscience opened this issue Jul 17, 2024 · 2 comments
Closed

Comments

@hummuscience
Copy link
Contributor

hummuscience commented Jul 17, 2024

I am on the main branch and using the dev pip install.

Running the training hydra script as usual and get the following error:

Screenshot from 2024-07-17 10-06-58

From what I understood, this error should not occur when using a fully supervised model (without pca or temporal losses).

This is related to the training of the Topvieemouse5k dataset, which has varying sizes: #158

Btw, this error does not occur on the dynamic_crop branch.

@hummuscience
Copy link
Contributor Author

I am a bit confused, as I was able to run this on the main brain a few days ago without getting this error. The only thing that changed was that I pulled the recent changes to the repo. So the error must have been quite recently, but I might be mistaken.

I switched to the dynamic_crop branch since it looks like this dependency on image size was merged there: #117

@themattinthehatt
Copy link
Collaborator

Oh yes, my apologies - ok I recommend switching to the unsuper-multiview branch as this (1) has the ability to train on multiple image sizes; and (2) does not run the validation data through the augmentation pipeline.

This branch is now PR #187 which will be merged into main in the next day or so!

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

No branches or pull requests

2 participants