Reported number of tiles is n-1

SWC-Advanced-Microscopy / BakingTray

Serial-section automated anatomy extension for ScanImage

https://bakingtray.mouse.vision/

GNU Lesser General Public License v3.0

10 stars 5 forks source link

Reported number of tiles is n-1 #262

Open raacampbell opened 4 years ago

raacampbell commented 4 years ago

The reported number of tiles to the acquisition log file is n-1. i.e. if a 10 by 8 grid is acquired the log file reports 79 tiles were acquired. This happens for acquisitions generated using both SIBT and dummyScanner

raacampbell commented 4 years ago

a31791b5fc64adc04c1d35967255de6f30e6431c

raacampbell commented 4 years ago

Changes involved in laying down all the tiles in the preview window (a8cd2f298fde3f4f68e4dc4b99918ea72bfde30d) have resulted in the number of tiles being n+1 again. So I revert this last commit. 7e1348cc5106ccf17c0fd0785141f9a34aa6d65c

Re-open this issue. Maybe there is a better solution to the above.

There was a bug introduced also by the position array being too large. This is fixed: d6c253d4dd3cbde3c91e5c0ed33893fcf4e9bde8

Let's see how it goes...