Fixing Chunking issues with codec

[edyoshikun]

This fixes #460.

The PR add the option that defaults for max chunk sizes of 500MB or max_chunk_size_bytes in mantis.cli.utils.create_empty_hcs().

Additionally the chunk size is set to Optional=None so that if None, the default chunks are 500MBs.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d31ad91) 9.45% compared to head (2674a42) 9.58%.

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #461      +/-   ##
========================================
+ Coverage   9.45%   9.58%   +0.13%     
========================================
  Files         29      29              
  Lines       4592    4599       +7     
========================================
+ Hits         434     441       +7     
  Misses      4158    4158              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[talonchandler]

Just tested this, and the issue #460 is still present.

This is not yet urgent for @dsundarraman's applications, so let's look at this closer after our deadlines.

[edyoshikun]

I looked closer to the issue and I realized that the issue is during the saving of the transfer functions. Here you would have to create_empty_zarr() or add the position.create_zeros() beforehand rather than allocating and saving the 'fov' directly into zarr here @talonchandler

[ziw-liu]

Magic numbers should either be derived from an expression or accompanied by a comment. For long numbers, the digits should be separated e.g. 2_147_483_647.