Task #12485 (closed)
Opened 10 years ago
Closed 10 years ago
Bug: reduce the number of companion files reported by VolocityReader
Reported by: | mlinkert | Owned by: | mlinkert |
---|---|---|---|
Priority: | critical | Milestone: | 5.1.0-m3 |
Component: | Bio-Formats | Version: | 5.0.2 |
Keywords: | n.a. | Cc: | johnbillheimer@…, jamoore, mtbcarroll |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | n.a. |
Sprint: | n.a. |
Description
See QA 9407. Large Volocity datasets throw Ice MemoryLimitExceptions? on import, as the "companion" binary data files can be very large. Adjusting getSeriesUsedFiles to identify most/all files as pixels files should fix this.
Change History (7)
comment:1 Changed 10 years ago by mlinkert
- Cc jamoore added
comment:2 Changed 10 years ago by jamoore
- Cc mtbcarroll added
- Priority changed from major to critical
Looking at that stacktrace, this is a matter of the number of files as opposed to their size, right? (Do you know how many?) If so, I'd think:
- short-term solution (5.0) : save the annotations links in batches
- middle-term solution (5.1): evaluate not creating these annotations. With FS, it's really unnecessary.
comment:3 Changed 10 years ago by mlinkert
Should be 994 files, ranging from 512 bytes to 61MB - guess I assumed size was the issue based upon the size of the largest files.
comment:4 Changed 10 years ago by jamoore
https://github.com/openmicroscopy/openmicroscopy/pull/2853 has been merged which should correct the most immediate error for 5.0.3. The Volocity fix itself and the middle-term removal of these companion files are still outstanding.
comment:5 Changed 10 years ago by jmoore <josh@…>
(In [e1589b08186a99a882a7a6145aeb09b472f5d39c/ome.git] on branch develop) Add batching to attachCompanionFilesToImage (See #12485)
comment:6 Changed 10 years ago by Josh Moore <josh@…>
(In [4652484245472e6d31e497ee320d14c318e86dd9/ome.git] on branch develop) Merge pull request #2867 from joshmoore/rebased/develop/12485-companion-loop
Add batching to attachCompanionFilesToImage (See #12485) (rebased onto develop)
comment:7 Changed 10 years ago by mlinkert
- Resolution set to fixed
- Status changed from new to closed
PR opened to address companion file classification: https://github.com/openmicroscopy/bioformats/pull/1267
#12519 opened to address the broader issue of companion attachment creation.
Full stack trace on import with 5.0.3-rc1: