NextEnd v5 trial release

[CDinger]

Martin,

NextEnd ran without any error. However, the game ended early and the recording followed the TV Guide schedule +2 minutes post padding. I expected it to end the recording early. A snippet of the NextEnd log below.

I need to look into modifying logrotate for NextPVR as my nrecord log is 1.9Mb so I moved my nrecord logs so I could zip and upload logs.

2020-01-18 12:32:59.036 [DEBUG][1] Schedule oid 1438 Status STATUS_IN_PROGRESS
2020-01-18 12:32:59.435 [DEBUG][1] LivePvrData.Data.Net.Responses.StatusResponse [Title=College Basketball, Status=Final, IsValid=True, IsActive=False, IsComplete=True, IsError=False, ]
2020-01-18 12:32:59.435 [DEBUG][1] Sports event has ended
2020-01-18 12:32:59.435 [DEBUG][1] Monitor status = final
2020-01-18 12:32:59.436 [DEBUG][1] Check to see if EP03275553.2188 stop exists
2020-01-18 12:32:59.437 [DEBUG][1] Extend file written -00:12:00.5641145
2020-01-18 12:32:59.438 [DEBUG][1] PowerManagement CancelWakeup (needs implementation)
2020-01-18 12:32:59.439 [DEBUG][1] Checking to cancel live recording
2020-01-18 12:32:59.439 [DEBUG][1] Exit NextEnd

Craig

[mvallevand]

Glad that is it working and thanks for test. You don't have the option --early so it stops one time.

Martin

[CDinger]

Ah, didn't realize there was an --early option. I revisited NextEnd on the Wiki. Does that mean I can I use both --early and --postgame 5 on the command line ? It is more important to me to extend when the game goes over rather than to end recording early if the game ends early.

Craig

[mvallevand]

Yes, basically early will stop it on time before the end time postgame gives you five minutes after the end. I can't remember if sub adds post padding or not, I don't use post game.

Martin

[pkscout]

OK, now that I have a stable, functional Docker NextPVR instance, I thought I'd look at NextEnd.  I saw in the first post that there are some other things to do for Docker, and I guess I'm the first Docker user, as it's not documented.  '-)  I'm also trying to figure out what is really required, so I thought I'd just try and ask all the questions at once.

1. Unzip attached zip file into a folder Plugins/NextEnd in the v5 user data folder.

Question: does this really go in /app/data/Plugins/NextEnd?  If so, that basically means building a custom Dockerfile to build a NextPVR image with NextEnd (or having to reinstall NextEnd after every container refresh).  Can this not go in /config/Plugins/NextEnd (which would survive a container refresh)?

2. If you aren't running in C:\Program Files\NPVR  edit NextEnd.runtimeconfiguration and change the C:\Program Files\NPVR to your binary folder keeping the trailing blanks

Question: Is that the path to where NextPVR lives?  Basically would that be /app/, right?  Could that just added to the additionalProbingPaths list, or do I really need to replace the C:/Program Files/NPVR entry?

3. On linux it might be tougher you also need to run this

Code:

sed -i 's/\"\/\"/\"\/opt\/nextpvr\/system\/\"/g' NextEnd.deps.json

I'm not 100% sure what this does, but I tried it on my OSX test machine so I could diff the file and see what changes where made.  sed errors out saying:

Code:

sed: 1: "NextEnd.deps.json": extra characters at the end of N command

Also you need to copy libe_sqlite3.so to system folder ie for 64 bit

sudo cp /opt/nextpvr/system/runtimes/linux-x64/native/libe_sqlite3.so /opt/nextpvr/system

I'm assuming that would need to be the /app/runtimes/linux-arm64/native/libe_sqlite3.so directory.  This is another one that would likely mean a custom Dockerfile to build a container so you don't have to reinstall NextEnd at every image refresh.



Is there other stuff I'm going to need to do?  Thanks.

[mvallevand]

For docker

The install folder for this should be on the host

/config/Plugins/NextEnd


The sed line is

Code:

sed -i 's/\"\/\"/\"\/app\/\"/g' NextEnd.deps.json

Change the end of NextEnd.runtime.json to point to the app folder, I will add this in the next update

Code:

"additionalProbingPaths": [
            "c:/Program Files/NPVR/",
            "/var/opt/nextpvr/",
            "/app/"

And prefix all calls in the sh file to the version of the runtime you are running to avoid copying the file

Code:

LD_LIBRARY_PATH=/app/runtimes/linux-x64/native/ dotnet /config/Plugins/NextEnd/NextEnd.dll ....options

Martin

[pkscout]

That seems to have worked.  Or at least NextEnd started and generated a log, which I take as a good sign.  So I know at least that the PostStartup.sh part is working.  BTW, it turns out my sed problem was that the BSD version of sed (which is what I was using on OSX to correct the file rather than doing it in the docker container) requires you to specify an extension for the original file when using -i.  Once I changed it to:

Code:

sed -i .org 's/\"\/\"/\"\/app\/\"/g' NextEnd.deps.json

it worked fine.

[pkscout]

That seems to have worked.  Or at least NextEnd started and generated a log, which I take as a good sign.  So I know at least that the PostStartup.sh part is working.  BTW, it turns out my sed problem was that the BSD version of sed (which is what I was using on OSX to correct the file rather than doing it in the docker container) requires you to specify an extension for the original file when using -i.  Once I changed it to:

Code:

sed -i .org 's/\"\/\"/\"\/app\/\"/g' NextEnd.deps.json

it worked fine.

I lied a little.  Turns out when I checked the nrecord log that NextEnd was crashing with an exception indicating it couldn't find the sql library.  After some poking around, it looks like you can't set LD_LIBRARY_PATH in the scripts in a Docker container.  It has to be set at run time.  So you have to run the container using something like this:

Code:

docker run -d --name nextpvr-pr \
--volume ~/Documents/NPVR/config:/config \
--volume ~/Documents/NPVR/recordings:/recordings \
--volume ~/Documents/NPVR/buffer:/buffer \
-e "SET_CONTAINER_TIMEZONE=true" \
-e "CONTAINER_TIMEZONE=Pacific/Honolulu" \
-e "LD_LIBRARY_PATH=/app/runtimes/linux-x64/native/" \
--publish 8866:8866 \
pkscout1/nextpvr-python3:prerelease

The important addition is:

Code:

-e "LD_LIBRARY_PATH=/app/runtimes/linux-x64/native/" \

Once I did that NextEnd runs without errors.

Some of the rest is different than sub's instructions because I'm using my Python3 build (and a NPVR pre-release that fixes a problem I was having with KNEWC in Kodi).  I included it because I know it works, and I didn't want to edit something from memory to try and make it look like sub's instructions without testing it.

[mvallevand]

@pkscout are you sure you had the syntax correct? I just tested here and my run looks ok, I turned off the stderr redirection and can see the debug output from NextEnd in the normal log.

2020-01-31 09:12:10.877 [DEBUG][9] Started recording (21952:7:/recordings/Scout & the Gumboot Kids/Season 01/Scout & the Gumboot Kids.S01E05.The Case of the Upside Down Smile.ts)
2020-01-31 09:12:11.049 [DEBUG][9] Starting: bash -c "'/config/scripts/ParallelProcessing.sh' '/recordings/Scout & the Gumboot Kids/Season 01/Scout & the Gumboot Kids.S01E05.The Case of the Upside Down Smile.ts' 4.1 21952 1546 'Scout & the Gumboot Kids'"
2020-01-31 09:12:11.745 [ERROR][117] NEXTPVR_INSTALLDIR /app/
2020-01-31 09:12:11.746 [ERROR][29] NEXTPVR_DATADIR /app/data/
2020-01-31 09:12:11.746 [ERROR][29] NEXTPVR_DATADIR_USERDATA /config/
2020-01-31 09:12:12.652 [ERROR][29] <Event>
...
2020-01-31 09:12:12.814 [ERROR][29] Not a sports recording


Martin

[pkscout]

@pkscout are you sure you had the syntax correct?  I just tested here and my run looks ok, I turned off the stderr redirection and can see the debug output from NextEnd in the normal log.

2020-01-31 09:12:10.877 [DEBUG][9] Started recording (21952:7:/recordings/Scout & the Gumboot Kids/Season 01/Scout & the Gumboot Kids.S01E05.The Case of the Upside Down Smile.ts)
2020-01-31 09:12:11.049 [DEBUG][9] Starting: bash -c "'/config/scripts/ParallelProcessing.sh' '/recordings/Scout & the Gumboot Kids/Season 01/Scout & the Gumboot Kids.S01E05.The Case of the Upside Down Smile.ts' 4.1 21952 1546 'Scout & the Gumboot Kids'"
2020-01-31 09:12:11.745 [ERROR][117] NEXTPVR_INSTALLDIR /app/
2020-01-31 09:12:11.746 [ERROR][29] NEXTPVR_DATADIR /app/data/
2020-01-31 09:12:11.746 [ERROR][29] NEXTPVR_DATADIR_USERDATA /config/
2020-01-31 09:12:12.652 [ERROR][29] <Event>
...
2020-01-31 09:12:12.814 [ERROR][29] Not a sports recording


Martin

I did not have the syntax right.  I'd never see a variable set on the same line as a command, so I just assumed that was a formatting error on the forum and put that on a separate line in the script.  And that apparently doesn't work.  The LD_LIBRARY_PATH declaration has to be on the same line as the command, and once I just copied and pasted what you had into my PostStartup.sh it worked.  So if I had just trusted that what you posted was actually right instead of over thinking it, I would have been fine.  ;-)