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

Update dataloaders.py remove float64 shapes #8865

Merged
merged 1 commit into from
Aug 4, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Aug 4, 2022

May help #8862

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhanced data loader efficiency by simplifying shape arrays.

πŸ“Š Key Changes

  • Removed explicit dtype=np.float64 specification from np.array(shapes) assignment.

🎯 Purpose & Impact

  • πŸš€ Purpose: The change is intended to simplify the data loading process by not explicitly defining the data type of the shape array. This could potentially lead to using a default data type that's more memory-efficient.
  • βœ… Impact: Users might experience improved memory usage and possibly faster data loading times when training their models, especially if the default data type is smaller than float64.

@glenn-jocher glenn-jocher self-assigned this Aug 4, 2022
@glenn-jocher glenn-jocher merged commit 84e7748 into master Aug 4, 2022
@glenn-jocher glenn-jocher deleted the fix/float64 branch August 4, 2022 15:06
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
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