harvard-lts / fits

File Information Tool Set
http://fitstool.org
GNU Lesser General Public License v2.1
92 stars 46 forks source link

MediaInfo for FITS version 1.5.5 does not work #385

Closed patelanm closed 9 months ago

patelanm commented 1 year ago

I am trying to run the FITS version 1.5.5. The MediaInfo configuration works completely fine for version 1.5.0, but when I upgraded the FITS version to 1.5.5 and tried to run the code I am seeing below error message.

<tool toolname="MediaInfo" toolversion="[could not launch tool]" status="failed"/>

It should show "did not run" instead of "failed" status.

I also compared all the files in both the versions 1.5.0 an 1.5.5 and everything looks identical.

Can someone please help me with this?

pwinckles commented 1 year ago

@patelanm

  1. Can you look for errors in the fits log file and post what you find?
  2. Is there a reason you upgraded to 1.5.5 instead of 1.6.0?
patelanm commented 1 year ago

@pwinckles I initially upgraded to 1.6.0 but wasn't able to disable TIKA so we decided to upgrade to 1.5.5 Also I am not able to find any log files. The business owner is requesting to disable it and show the status "did not run". Is there a way to do that?

pwinckles commented 1 year ago

@patelanm Yes, but you haven't provided enough information in either of your tickets for us to help you. Please provide your config in https://github.com/harvard-lts/fits/issues/384.

patelanm commented 1 year ago

Here is my fits config file content: fits_config.txt

pwinckles commented 1 year ago

@patelanm I would rather help you get 1.6.0 working than 1.5.5. Here's the relevant file in regards to Tika in your configuration:

<tool class="edu.harvard.hul.ois.fits.tools.tika.TikaTool" exclude-exts="jar,avi,mov,mpg,mpeg,mkv,mp4,mpeg4,m2ts,mxf,ogv,mj2,divx,dv,m4v,m2v,ismv,tif,tiff" classpath-dirs="lib/tika"/>

Now, which file extension were you attempting to exclude from Tika?

patelanm commented 1 year ago

Below is the response I got from my product owner:

Our FITS is only pointing to the IMAGES folder to look for image files, i.e., there will be no txt, doc or video files shown in the folder at all. Do we still have to list all those file extensions for exclusion? Currently, we only have .tif or .tiff but we had .jpeg in the past. We may have .jp2 in the future.

If we have to list all of them regardless, then we will want them to provides us a complete file extension list Tika is programmed to look for.

pwinckles commented 1 year ago

@patelanm Okay, so you either need to add the .jpeg and .jp2 extensions to the exclude-exts lists on that config line, or, if you don't want Tika to run ever, remove the line entirely.

dchandekstark commented 9 months ago

I can confirm this problem as well, at least on Debian bookworm. Installing the mediainfo OS package and removing tools/mediainfo from FITS makes no difference.

pwinckles commented 9 months ago

@dchandekstark are you using version 1.5.5? If so, why?

dchandekstark commented 9 months ago

@pwinckles Yes, using 1.5.5. I had some issue with 1.6.0 but TBH can't remember now ... will give it a try.

pwinckles commented 9 months ago

@dchandekstark the latest version of MediaInfo does not work with pre-1.6.0 releases. I strongly recommend using 1.6.0. If you have issues with it, open a new ticket.