Skip to content

Pull request for TIKA-1798 Parser for Video Similarity using PooledTimeSeries metric contributed by Aditya Dhulipala and Chris Mattmann. - #64

Closed
chrismattmann wants to merge 20 commits into
apache:trunkfrom
chrismattmann:trunk
Closed

Conversation

@chrismattmann

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs to be generalized.

@chrismattmann

Copy link
Copy Markdown
Contributor Author

ok @cafed00d4j I significantly cleaned this up and got it working! Great initial work!

Some comments:

  1. I thought we discussed using a table here. Since you have # of rows and # of cols present in the div, why not simply use tables and columns here to represent the OF and OG? (I'll fix this now).
  2. you don't have an ending div on your XHTML (I'll add a fix for this now).
  3. You don't identify in the XHTML what is the OF and what is the OG (I'll fix this now).
  4. We should probably start out with an h3 of the file name to start out the display.
  5. I get a whole bunch of NaNs before the XHTML output. Not sure where those are from but they are printed to stdout.

@adi-dhulipala

Copy link
Copy Markdown
Contributor

Hi Prof. Chirs,

Thanks for the feedback.. Please check my responses below.

  1. I thought we discussed using a table here. Since you have # of rows and # of cols present in the div, why not simply use tables and columns here to represent the OF and OG? (I'll fix this now).
    ------>
    In one of the progress reports I think you mentioned we can use "ol li --- li ol" sequences instead
    The OF and HOG are series of 1D vectors instead of series of 2D matrices as I originally thought.
  2. you don't have an ending div on your XHTML (I'll add a fix for this now).
  3. You don't identify in the XHTML what is the OF and what is the OG (I'll fix this now).
    ------>
    Sorry about that. I think I missed it
  4. We should probably start out with an h3 of the file name to start out the display.
    ------>
    Understood. Will fix it now
  5. I get a whole bunch of NaNs before the XHTML output. Not sure where those are from but they are printed to stdout.
    ------>
    Yes. These are the meanChiSquare distance measures of the FeatureVectors. Since, we're running this on a single file, we only have one set of FeatureVectors.
    When PoT is run on a collection of video files, we extract each video's OF and HOG and then calculate the meanSquareDist across all of them.
    But this may not be within the scope of a Tika parser. Maybe it should be included in the wrapper program that calls Tika on a set of videos. Much like the implementation of tika-img-similarity github project. Do you think that's right?

@chrismattmann

Copy link
Copy Markdown
Contributor Author

@cafed00d4j i don't get the response to 1.i. - if OF and HOG Are 1D vectors why is there a num rows, num cols?

@adi-dhulipala

Copy link
Copy Markdown
Contributor

I couldn't think of better terms for them.

The numRows is number of time-series in the video. This proportional to the numFrames in the video. But it's not equal to it. It depends on the how the underlying computeFarnebackFlow() and computeGradeient() operations behave. but the general term the paper uses is time-series and each of the rows is a unit of time-series

The numCols is a constant. Of size 200. It's the vector size. It's got by applying some pooling operators on the flowVectors and gradientHistorgrams. (It's initialized around line 412. There's also a comment there about the chosen size)

So, each row in the output is a PoT vector of that particular time-serie.

@chrismattmann

Copy link
Copy Markdown
Contributor Author

Thanks @cafed00d4j so I think we can still use a table here and just do rowspan to take care of non uniform sizes - colspan isn't needed since it's always 200 right?

@adi-dhulipala

Copy link
Copy Markdown
Contributor

Hi Prof.

No problem!

Yes. I think that can omitted. But we can leave it in for completeness it's that's important.

… code. 2. Use table instead of vector. 3. Provide of and og vecSize and frameSize in metadata. 4. add of and og computations. 5. fix XHTML and disable of and og table headings.
@asfgit asfgit closed this in def27b6 Nov 27, 2015
tballison pushed a commit to tballison/tika that referenced this pull request Feb 26, 2016
…metric contributed by Aditya Dhulipala and Chris Mattmann this closes apache#64.

git-svn-id: https://svn.apache.org/repos/asf/tika/trunk@1716927 13f79535-47bb-0310-9956-ffa450edef68
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