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

Bnb/training with obs #252

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

Bnb/training with obs #252

wants to merge 25 commits into from

Conversation

bnb32
Copy link
Collaborator

@bnb32 bnb32 commented Dec 27, 2024

Added observation member to Sup3rDataset and minor modifications to samplers / batch queues to enable additional observation based loss term in training. Split sampling and enqueuing in batch handling which seems to have at least partially resolved the max_workers > 1 issue.

@bnb32 bnb32 force-pushed the bnb/training_with_obs branch from 6c036e0 to b1cb942 Compare December 27, 2024 16:41
@bnb32 bnb32 force-pushed the bnb/training_with_obs branch from ee31120 to 046a279 Compare December 29, 2024 23:45
…d dequeueing since this would cast to tensors.
@bnb32 bnb32 force-pushed the bnb/training_with_obs branch from 046a279 to f6f3e40 Compare December 30, 2024 00:12
@bnb32 bnb32 force-pushed the bnb/training_with_obs branch from bd493b1 to 7897998 Compare January 2, 2025 20:05
@bnb32 bnb32 force-pushed the bnb/training_with_obs branch from 2b9c0d0 to e1ee79a Compare January 3, 2025 23:11
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