-
Notifications
You must be signed in to change notification settings - Fork 102
🐛 Fix dictionary changed size Error
#626
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
Conversation
- Fix `dictionary changed size` error in 08-nucleus-instance-segmentation and 09-multi-task-segmentation - Changing num_loader_workers=0 and num_postproc_workers=0 fixes the error on Colab.
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
09-multi-task-segmentation fails to load tmp\sample_tile_results\0.raw.0.npy still fails which is not saved. |
Codecov Report
@@ Coverage Diff @@
## develop #626 +/- ##
========================================
Coverage 99.84% 99.84%
========================================
Files 64 64
Lines 7305 7305
Branches 1405 1405
========================================
Hits 7294 7294
Misses 4 4
Partials 7 7 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
I noticed that you have set the worker subprocesses to 0 in the notebooks. However, is this really addressing the root cause of the issue? I believe we should be able to use more workers in the notebooks. Is this just a temporary solution? |
I have tested this locally with multiple Python versions on Windows and Ubuntu with num_workers=4, 16 and it works fine. Something failing with multiprocessing on Colab. |
dictionary changed size errordictionary changed size Error
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
for more information, see https://pre-commit.ci
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
… into bug-fix-dictionary-resize-error # Conflicts: # examples/09-multi-task-segmentation.ipynb
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
for more information, see https://pre-commit.ci
…-dictionary-resize-error
- ### Bug Fixes and Other Changes - Fix dictionary changed size Error #626 (#605) Full Changelog: v1.4.0...v1.4.1 Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- ### Bug Fixes and Other Changes - Fix dictionary changed size Error #626 (#605) Full Changelog: v1.4.0...v1.4.1
dictionary changed sizeerror in 08-nucleus-instance-segmentation and 09-multi-task-segmentation