Add MRC file writing support

[Pradyumn-cloud]

Fix Issue - #108

Add MRC file writing support

• Implement MRC.write() method
• Add Grid.export() integration for MRC format
• Added Test
• Preserve header information and handle edge cases

[codecov]

Codecov Report

❌ Patch coverage is 93.44262% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.13%. Comparing base (abd0feb) to head (18a2192).

Files with missing lines	Patch %	Lines
gridData/mrc.py	92.30%	0 Missing and 4 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
+ Coverage   87.70%   88.13%   +0.43%     
==========================================
  Files           5        5              
  Lines         748      809      +61     
  Branches       97      105       +8     
==========================================
+ Hits          656      713      +57     
  Misses         56       56              
- Partials       36       40       +4     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[orbeckst]

Looks good at first glance but I am not an MRC expert. I'll see if I can get someone else to also have a look.

In the meantime, please also

• update your branch with the latest changes on master (just did that)
• add yourself to AUTHORS
• add an entry to CHANGELOG
• check your coverage and add tests where necessary

[orbeckst]

@marinegor do you have some experience with MRC files and would you be able to have a very brief look at this PR?

[marinegor]

Hi, thanks for the PR!

I've added couple of comments regarding self.header vs self._mrc_header, and regarding using with open(...).

Also, perhaps it's a good idea to add a small test MRC (say, 4x4x4 with random numbers) file with/without a header that tests things? This way we can actually track regressions if something changes in mrcfile itself, while a roundtrip test won't check that.

[orbeckst]

Some minor fixes required and please address @marinegor 's comments. That should also help with test coverage.

[orbeckst]

Sorry it took me a while to get back to the PR. Thank you for addressing all remaining issues and improving the tests. This is good to go and I'll merge.

[orbeckst]

@marinegor could you please have a quick look and confirm that all your concerns have been addressed? Thanks!