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.

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

Config pin_memory Option #67

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Config pin_memory Option #67

merged 1 commit into from
Sep 3, 2024

Conversation

JSabadin
Copy link
Contributor

@JSabadin JSabadin commented Aug 28, 2024 •
edited by kozlov721
Loading

Added the trainer.pin_memory option to the config. Defaults to True.

@github-actions GitHub Actions
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
5336 4498 84% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
luxonis_train/core/core.py 80% 🟢
luxonis_train/utils/config.py 92% 🟢
TOTAL 86% 🟢

updated for commit: 8893d10 by action🐍

@github-actions GitHub Actions
Copy link

Test Results

 2 files   2 suites   26m 46s ⏱️
45 tests 45 ✅ 0 💤 0 ❌
92 runs  91 ✅ 1 💤 0 ❌

Results for commit 8893d10.

@kozlov721 kozlov721 added the enhancement New feature or request label Aug 28, 2024
@kozlov721 kozlov721 changed the title pin_memory Config pin_memory Option Sep 3, 2024
@kozlov721 kozlov721 merged commit 5acabfc into dev Sep 3, 2024
6 checks passed
@kozlov721 kozlov721 deleted the feat/pin-memory branch September 3, 2024 19:32
kozlov721 pushed a commit that referenced this pull request Oct 9, 2024
@JSabadin @kozlov721
Config pin_memory Option (#67)
eeb4f45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@klemen1999 klemen1999 klemen1999 approved these changes

@kozlov721 kozlov721 kozlov721 approved these changes

@tersekmatija tersekmatija Awaiting requested review from tersekmatija

@conorsim conorsim Awaiting requested review from conorsim

Assignees

@JSabadin JSabadin

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JSabadin @kozlov721 @klemen1999