Yes there have been several fixes recently. We are aiming to make the new rewriter the default for everyone soon. If you experience issues let us know.
Tests went great last night on discovery and history channels prime time lineup.
Programs had full length and i didnt notice any dropped frames.
We are continuing to have confidence that the new rewriter is handling all of the cases we have looked at better than the old one. Please let us know if you run into any issues with the new one that we need to look into further.
It's been working fine so far.
When @tmm1 mentioned that you would soon be making it the default I started using it. I have noticed no issues to report. It’s working great for me on my admittedly generic Synology/YoutubeTv setup. Hopefully once this becomes a default maybe it will make things easier with the CC for Pluto dream.
After enabling this, I'm no longer getting the MTS stats in the log for recordings, like this
Can they be added back.
highest_pts, saw_pcr, skipped and unhandled_packets are useful.
Not really as such. The new rewriter doesn't operate the same way as the old one, so skipped=
and unhandled_packets=
aren't a thing that is meaningful in the new system.
I can look at adding back highest_pts=
, but the meaning in the new system is somewhat different from what it was before...
Thanks. Mainly useful for interrupted recordings so I can quickly locate the spot in my video editor where Channels DVR stitched the stream back together.
saw_pcr=false meant something was changing the PMT to say the PCR was in a PID when there was actually no PCR in the stream EXPERIMENTAL: New MPEG-TS Rewriter for HLS (TVE) - #84 by chDVRuser
I enabled this 2 weeks ago on all four DVR servers and haven't seen any issues with recordings or Live.
Using TVE and M3U sources.
Will post if any issues with these 21 one hour recordings this week (Shark Week on Discovery).
Day one 4 hrs of back-back recordings, all good
Day two 3 hrs of back-back recordings, all good
Day three 3 hrs of back-back recordings, all good
Day four 3 hrs of back-back recordings, all good
Day five 3 hrs of back-back recordings, all good
And after recording, updated to
and
just to make sure I'm testing w/latest version
Day six 3 hrs of back-back recordings, all good
Day seven 2 hrs of back-back recordings, all good
All 21 hours without any issues.
Pausing the pass until next year.
Thanks @eric for the work on the new rewriter!
After upgrading to v2023.07.27.2357 with the MPEG-TS Rewriter enabled, Newsmax (channel 6764) is now unwatchable as it stutters terribly. It was smooth as butter in the prior beta version…
This latest update should resolve the issues you saw.
That did the trick! Thanks so much Eric. Have a blessed weekend my friend…
After investigating further I was able to figure out how wire up the rewriter statistics for the new rewriter. Some of the values will always be 0
with the new system (skipped
, unhandled_packets
, invalid_pts
, invalid_dts
), but the other ones that sounded like the most important ones will be working.
Thank eric, looks great
2023/07/30 17:00:29.526430 [MTS] Statistics for "TV/Fixer Upper/Fixer Upper S05E18 2018-04-03 Rock Star Renovation 2023-07-30-1646.mpg": skipped=0 unhandled_packets=0 discontinuity_detected=15 transport_errors=0 invalid_pts=0 invalid_dts=0 saw_pcr=false saw_pmt=true highest_pts=856.434744
highest_pts=856.434744
Exactly correct!
Even though it shows saw_pcr=false
, the recording does have a PCR in the right place, the video PID. Not sure if you're rewriting the PCR?
Yes, we're adding a synthetic PCR if one hasn't been seen in 100ms to conform to the MPEG-TS spec. That saw_pcr=false
is indicating that we did not see one in the original stream.
The new MPEG-TS rewriter is now default for everyone. Thanks to everyone who helped with the testing!
“The old one….” Think is what u ment to say in the release notes is the one that’s available in experimental settings
Is this update going to automatically remove the Check Mark to enable the writer ? if not we will be using the old.