Skip to content

Revert "Purge opencensus libraries from the BOM"#311

Merged
ctrueden merged 1 commit into
scijava:masterfrom
cmhulbert:revert-opencensus-exclusion
Jul 17, 2026
Merged

Revert "Purge opencensus libraries from the BOM"#311
ctrueden merged 1 commit into
scijava:masterfrom
cmhulbert:revert-opencensus-exclusion

Conversation

@cmhulbert

Copy link
Copy Markdown
Contributor

This reverts commit 7a2c1b3.

It was previously recognized that excluding io.opencensus broke n5-google-cloud. This was reverted in Fiji, but not in pom-scijava. As such, upgrading n5-universe or n5-google-cloud to 45.0.0 currently fails with the following style of errors:

Error: org.janelia.saalfeldlab.n5.universe.storage.zarr.zarr3.Zarr3GoogleCloudFactoryTest$ZarrGoogleCloudMockTest.testWriteReadIntBlock
Error: Run 1: Zarr3GoogleCloudFactoryTest$ZarrGoogleCloudMockTest.:64 » NoClassDefFound io/opencensus/trace/propagation/TextFormat$Setter

44.0.0 still works as expected.

@cmhulbert

Copy link
Copy Markdown
Contributor Author

The comment linked about was regarding adding io.opencensus back to the Fiji update site, but it the most recent Fiji-latest it is still the case that attempting to open a google-cloud backed project with N5 throws an exception. My guess is even if Fiji allows it in the update sites, since the pom-scijava excludes it, it never attempts to pull it in as a dependency

@ctrueden
ctrueden merged commit 640a73e into scijava:master Jul 17, 2026
1 check passed
@cmhulbert
cmhulbert deleted the revert-opencensus-exclusion branch July 20, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants