Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update to scijava 34.1.0 and use Location API #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update to scijava 34.1.0 and use Location API #7

wants to merge 3 commits into from

Conversation

ecmagnuson
Copy link
Contributor

@ecmagnuson ecmagnuson commented Apr 6, 2023 •
edited
Loading

closes #5

This updates the tutorials to use the scijava location api.

Some things to note:

  1. I made a class FakeTutorialImages to use the builder for getting fake image locations. There is a method sampleIndexedImage where I'm not certain if that is the best name for the method, but it is used in T2bTypedComponents.java to see if the Color table isn't null.
  2. In T1OpeningOMEXML the class runs exactly the same, but for some reason there is an error when running. This is a log error at the top and doesn't kill the program
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

When it calls T1dSavingImagePlanes.main(); this error gets thrown at creating a new SCIFIO instance, which I am not certain on currently.

@ecmagnuson ecmagnuson requested a review from hinerm April 6, 2023 21:16
@hinerm
Copy link
Member

hinerm commented Jun 19, 2023

2. In T1OpeningOMEXML the class runs exactly the same, but for some reason there is an error when running. This is a log error at the top and doesn't kill the program

try adding:

     <dependency>
      <groupId>ch.qos.reload4j</groupId>
      <artifactId>reload4j</artifactId>
    </dependency>

as a dependency and see if this warning goes away. It's probably because we (appropriately) stopped shipping a logging implementation (and the whole log4j security vulnerability thing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hinerm hinerm Awaiting requested review from hinerm

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update scifio-tutorials to new Location API
2 participants
@ecmagnuson @hinerm