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

Observed a "SyntaxError: Unexpected end of JSON input" for the mpd that contains Role and Accessibility elements when we enable the "DVB (2018 xsd)" option on the conformance tool. #707

Open
Ramajayadevi opened this issue Aug 13, 2024 · 10 comments

Comments

@Ramajayadevi
Copy link

Ramajayadevi commented Aug 13, 2024

Below is the mpd we tried to validate in the dash conformance tool when we enable "DVB (2018 xsd)" option , but the tool is not able to parse the mpd and says "SyntaxError: Unexpected end of JSON input" error.

When we dont enable "DVB (2018 xsd)" option while validating the mpd in the conformance tool, then the mpd validation is passed without syntax error.
visually_impaired_with_both_mix_type0_1.txt

Here is the conformance tool we used to validate this mpd:
https://conformance.dashif.org/

When we refer the issues reported to the dash forum, found this bug which is related to the same error, but this error is not resolved.
#677

Could you please check and provide the feedback.

@Ramajayadevi
Copy link
Author

Ramajayadevi commented Aug 13, 2024

image

here is the image which tells the option we enabled while validating the attached mpd

@Phencys
Copy link
Collaborator

Phencys commented Sep 3, 2024

This issue was triggered by a divergence of options between the CLI and REST api's, in combination with the upgrade to PHP8.x

Should be resolved on https://staging.conformance.dashif.org/

@kjerbi
Copy link

kjerbi commented Sep 13, 2024

Hi @Phencys ,
Thanks @Ramajayadevi for raising the issue.
I am still seeing the problem even with the staging link. I confirm the issue appears when using dvb2018.
Any updates?
Manifest attached
manifest_dvb2018.zip

Many thanks in advance

@Ramajayadevi
Copy link
Author

Hi @kjerbi,

Yes, Am also still seeing the issue with the staging link which was shared here.

Thanks,
Ramajayadevi.

@Phencys
Copy link
Collaborator

Phencys commented Sep 16, 2024

Hello @kjerbi and @Ramajayadevi

Thanks for the update on the issue and the extra manifest to debug.

Apparently there were a lot of surpressed warnings in the code, and some code that used to be valid before the PHP8 upgrade now failed silently (but triggering other issues)

The version on staging supports both updated manifests now. Can either of you confirm?

@Ramajayadevi
Copy link
Author

Hi @Phencys ,

Thanks for the response.

Still with the same staging tool, am seeing an error.

Thanks,
Ramajayadevi.

@Phencys
Copy link
Collaborator

Phencys commented Sep 16, 2024

Some extra context on the bug:
This message is given whenever the PHP code crashes. Independent of where or how, if there is an issue with the PHP code itself, this message is displayed in the interface. Because PHP code is only 'really' checked at runtime, a lot of the issue only arise if we have specific enough scenario's.

Looking at the logs on the staging environment, I can trace back one error, but it seems related to segment analysis.

I have tried to fix it, but this is quite tricky without the actual segments. @Ramajayadevi can you please check for me

  1. If the analysis works for you when the segments are not validated
  2. If the analysis now works with segment validation enabled?

If step 2 is still a no, can you possibly provide us a couple of the segments in the stream, to allow better debugging?

@kjerbi
Copy link

kjerbi commented Sep 16, 2024

@Phencys
many thanks for th quick reaction!
I confirm my manifest now does not have the "unexpected end of json" issue.

Kind regards
Khaled

@Phencys
Copy link
Collaborator

Phencys commented Sep 16, 2024

I have created a new issue (#708) to track progress on the bigger picture behind this issue.

It will be picked up as part of the next release.

@Ramajayadevi
Copy link
Author

Ramajayadevi commented Sep 17, 2024

Hi @Phencys ,

Thanks for the response.

I have checked,

Without enabling the "Segment validation" option on the staging link and i can see my mpd has been validated without syntax error.
With enabling the "Segment validation" option, i can see the same syntax error.

Thanks,
Ramajayadevi.

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

No branches or pull requests

3 participants