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.

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

Odd behavior in Text vs XML handler for WAVE-hul #881

Open
thorsted opened this issue Sep 13, 2023 · 5 comments
Open

Odd behavior in Text vs XML handler for WAVE-hul #881

thorsted opened this issue Sep 13, 2023 · 5 comments
Assignees
Labels
bug A product defect that needs fixing P2 Medium priority issues to be scheduled in a future release
Milestone

Comments

@thorsted
Copy link

I noticed while using the WAVE-hul 1.8.2 I get different results with a text based handler than I do with XML. Test output shows time duration correctly, but XML does not, only zeros.

  Duration:
   FrameCount: 30
   TimeBase: 1000
   VideoField: FIELD_1
   CountingMode: NTSC_NON_DROP_FRAME
   Hours: 0
   Minutes: 1
   Seconds: 30
   Frames: 0
   Samples: 
    SampleRate: S44100
    NumberOfSamples: 0
<aes:timeRange>
         <tcf:startTime tcf:frameCount="30" tcf:timeBase="1000" tcf:videoField="FIELD_1" tcf:countingMode="NTSC_NON_DROP_FRAME">
          <tcf:hours>0</tcf:hours>
          <tcf:minutes>0</tcf:minutes>
          <tcf:seconds>0</tcf:seconds>
          <tcf:frames>0</tcf:frames>
         </tcf:startTime>
         </aes:timeRange>

output.zip

@carlwilson carlwilson added bug A product defect that needs fixing P2 Medium priority issues to be scheduled in a future release labels Sep 14, 2023
@carlwilson
Copy link
Member

Thanks @thorsted I suspect that there might be a similar issue to this logged. We're doing some triage work in the next 2 weeks and will try to group this if possible.

@thorsted
Copy link
Author

Thanks Carl, I noticed it would work when I used a Broadcast wave with TimeReference added, but never could get the XML to give duration otherwise. Let me know if you need samples.

@samalloing
Copy link
Collaborator

Hi @carlwilson and @thorsted

After a quick look, I expect that duration is processed in the TextHandler (

) and in the XMLHandler this is not:

Sam

@thorsted
Copy link
Author

@carlwilson Just checking in on this issue.

@samalloing
Copy link
Collaborator

Hi @carlwilson

Can this issue be added to the jHove 1.32 release?

Thanks

Sam

@carlwilson carlwilson self-assigned this Aug 22, 2024
@carlwilson carlwilson added this to the JHOVE 1.34 milestone Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A product defect that needs fixing P2 Medium priority issues to be scheduled in a future release
Projects
None yet
Development

No branches or pull requests

3 participants