8000 GUI sets `pad_to_stride` to 16 by default · Issue #2163 · talmolab/sleap · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

GUI sets pad_to_stride to 16 by default #2163

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.

Sign up for GitHub

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

Jump to bottom
Open
4 tasks
gitttt-1234 opened this issue Apr 9, 2025 · 0 comments · May be fixed by #2164
Open
4 tasks

GUI sets pad_to_stride to 16 by default #2163

gitttt-1234 opened this issue Apr 9, 2025 · 0 comments · May be fixed by #2164
Labels
bug Something isn't working

Comments

@gitttt-1234
Copy link
Contributor

Bug description

When training a model through the GUI, the preprocessing configuration sets the pad_to_stride parameter to a default value of 16. However, this value should ideally match the max_stride specified in the model configuration to ensure alignment. Currently, pad_to_stride is only updated to max_stride if it is explicitly set to None here

sleap/sleap/nn/training.py

Lines 1021 to 1023 in 8228cd2

if self.config.data.preprocessing.pad_to_stride is None:
self.config.data.preprocessing.pad_to_stride = self.model.maximum_stride

Since the GUI always defaults to 16, this override does not occur—even when a different max_stride is required—potentially leading to inconsistencies during training.

Expected behaviour

pad_to_stride should automatically match max_stride from the model config unless explicitly set.

Actual behaviour

The GUI always sets pad_to_stride to 16, leading to mis-alignment with max_stride.

Your personal set up

  • OS:
  • Version(s):
Environment packages 
# paste output of `pip freeze` or `conda list` here
Logs 
# paste relevant logs here, if any

Screenshots

How to reproduce

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error
@gitttt-1234 gitttt-1234 added the bug Something isn't working label Apr 9, 2025
@gitttt-1234 gitttt-1234 changed the title Pad_to_stride is always set at 16 GUI sets pad_to_stride to 16 by default Apr 9, 2025
@gitttt-1234 gitttt-1234 linked a pull request A 70F8 pr 9, 2025 that will close this issue
Open
11 tasks
@gitttt-1234 gitttt-1234 linked a pull request Apr 9, 2025 that will close this issue
Fix default for pad_to_stride parameter #2164
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix default for pad_to_stride parameter talmolab/sleap
1 participant
@gitttt-1234
0