Remove unused code for zarr v3 specific handling from test_zarr_avg_merger

[Borda]

This pull request simplifies the handling of compressor and codec arguments in the test_zarr_avg_merger_patches test by removing the logic specific to zarr v3. Now, compressor-related arguments are only processed for zarr v2, which streamlines the code and reduces complexity.

Compressor and codec handling simplification:

• Removed special handling for compressor, value_compressor, and count_compressor arguments when using zarr v3; these arguments are now only processed for zarr v2, eliminating the conversion to codec lists for v3 and related code paths.

Description

according to the coverage report it is unused code: https://app.codecov.io/gh/Project-MONAI/MONAI/blob/dev/tests%2Finferers%2Ftest_zarr_avg_merger.py

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[coderabbitai]

📝 Walkthrough

Walkthrough

This change modifies zarr compressor handling in test code. The conversion from compressor to codecs now only applies for zarr v2, while v3 uses codecs directly. The refactoring removes 32 lines of code that previously contained zarr v3-specific codec mappings, eliminating the separate v3 conversion branch for compressor, value_compressor, and count_compressor arguments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	Description covers the change with motivation (coverage report) and includes the required types-of-changes checklist.
Title check	✅ Passed	The title accurately describes the main change: removing zarr v3 specific handling code from the test file.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This pull request removes unused code that was intended to handle zarr v3 specific conversion of compressor arguments to codecs. The removed code was dead because test cases already handle version detection at test case definition time, providing either compressor arguments for zarr v2 or codecs arguments for zarr v3.

Changes:

• Removed conditional logic that converted compressor, value_compressor, and count_compressor arguments to codec lists when zarr v3 is active
• Simplified the code to only process compressor arguments for zarr v2

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.