accessing Person:sage.Person via sagex?

[davidk21770]

My wiz.bin recently went through an explosion. No matter how far I went back, it grew quickly >250m in a few days, upwards to 950m when I was away for vacation over Christmas. My wiz has had small issues for years (shows that wouldn't stay watched), but I suspect there was a change in the latest sage 9 that made these much worse. To me, it looks like a garbage collection process does not like something in this wiz. But this wiz has been around since 2008 give or take It's the agent table that grows. People.

I tried to do the restart as described but found the xml tools to have issues. Favorites did not have their start/stop delays; watched was forgotten; if I tried to include watched I ended-up with duplicate shows. I don't remember if I even tried playlists. But then while I was pruning add-ons and updating jetty, xml dumps from the web interface broke <sigh>.

So I delved into sagex and hacked myself to a new wiz.bin. Which is stable at just shy of 50m. <Yay!> (Over 8,000 shows).

Where I need help is that I can not seem to add a favorite person. The Sage API - 9.2.6.1 says that I can use person: person-string-url-ized. This works fine with playlists, mediaids, and airingids -- but not here. It gives me a "Result: true" response, but no person is added to the favorite. Either using AddFavorite or SetFavoritePerson.

Here is an attempt to add Joss Whedon to my HD movie channels.

curl --no-progress-meter "http://...@192.168.1.99:8080/sagex/api?c=AddFavorite&filter=FavoriteID|FavoriteTitle|FavoritePresentationName&1=&2=true&3=true&4=&5=&6=person:Joss%20Whedon&7=&8=&9=&10=&11=&12=EPIXHD%3bHBOHD%3bIFCHD%3bSHOWHD%3bTCMHD%3bTMCHD&13=&14=&start=0&size=5000&encoder=xml"

It does create a favorite -- with only the channels selected.
This works fine for adding show favorites (&1=)
(And I've tried it encoding the ':' also.)

Am I missing something?

[stuckless]

It would appear that person mapping is not implemented.

https://github.com/stuckless/sagetv-...Util.java#L188

[davidk21770]

Originally Posted by stuckless

It would appear that person mapping is not implemented.

https://github.com/stuckless/sagetv-...Util.java#L188

Thank you! I will stop trying different combinations of possible errors .

[jusjoken]

Originally Posted by stuckless

It would appear that person mapping is not implemented.

https://github.com/stuckless/sagetv-...Util.java#L188

I have been working with this source code lately so I could take a look at this and see if I could add it in.

K

[stuckless]

Originally Posted by jusjoken

I have been working with this source code lately so I could take a look at this and see if I could add it in.

K

If there's a GetPersonById and IsPersonObject apis in core SageTV then it will be pretty each to update that function (and the one after it) and then "person:somePersonId" mappings would work.

[jusjoken]

Originally Posted by stuckless

If there's a GetPersonById and IsPersonObject apis in core SageTV then it will be pretty each to update that function (and the one after it) and then "person:somePersonId" mappings would work.

I figured that part out now but the generated api does not include the PersonAPI. I am working through the code and it seems the generator build from the HTML docs located here...

http://download.sage.tv/api/sage/api/

So for example it build the PlaylistAPI from here...

http://download.sage.tv/api/sage/api/PlaylistAPI.html

But the is no PersonAPI.html so it does not generate the PersonAPI.

Is there a way to work around that or do I need to ask Jeff to create those docs ?

K

[stuckless]

Originally Posted by jusjoken

I figured that part out now but the generated api does not include the PersonAPI. I am working through the code and it seems the generator build from the HTML docs located here...

http://download.sage.tv/api/sage/api/

So for example it build the PlaylistAPI from here...

http://download.sage.tv/api/sage/api/PlaylistAPI.html

But the is no PersonAPI.html so it does not generate the PersonAPI.

Is there a way to work around that or do I need to ask Jeff to create those docs ?

K

Other than manually creating it, I don't think there is an easy way. I'm surprised there is no Person.html javadocs.

[jusjoken]

I have updated sagex to include the PersonAPI but not sure I know how to test this api.

@davidk21770 If you have a test I can perform let me know or pm me and I can find a way to get you the new jar to test before I release an update.

K

[Narflex]

It took some digging around...but I finally found the script that I had which generated the javadocs from the code. You should be able to use this to generate the new PersonAPI. If you get this generated, you can just email me the zip file for it and I can then also upload it to the server so it's there as well.

#!/bin/bash

cd `dirname $0`
pwd


BASEDIR=../..
SRCDIR=${BASEDIR}/java
OUTDIR=temp-doc

if [ "${JDK_HOME}" ] ; then
  export PATH=${PATH}:${JDK_HOME}/bin
fi

for testprog in javac javadoc java ; do
  if [ -z `which $testprog` ] ; then
    echo $testprog not on search path - add JDK to PATH or run with JDK_HOME set
    exit 1;
  fi
done


rm -fr temp-doc temp-src temp-bin

# Compiling some of the required classes -- ignore any errors
mkdir temp-bin

# exit on error from here on...
set -e
javac -d temp-bin -sourcepath ${SRCDIR} \
   ${SRCDIR}/tv/sage/util/StudioAPIProcessor.java

echo ""
echo run API processor to build dummy java files from api source files
mkdir temp-src
java -cp temp-bin \
   tv.sage.util.StudioAPIProcessor \
   ${SRCDIR}/sage/api/ \
   temp-src/sage/api

mkdir -p temp-src/sage/media/rss
cp -r ${SRCDIR}/sage/media/rss temp-src/sage/media/

# remove ReflectionAPI as that is indended to be hidden
rm temp-src/sage/api/ReflectionAPI.java


echo ""
echo "Building Javadocs"

# get version num from source Version.java
eval $(grep -e '_VERSION *=' ${SRCDIR}/sage/Version.java |\
       sed -e 's/.* \(.*_VERSION\) *= *\(.*\);/\1=\2/')

VERSION=${MAJOR_VERSION}.${MINOR_VERSION}
DOCTITLE="SageTV<sup><font size='-2'>TM</font></sup> V${VERSION} API Specification"
HEADER="<b>SageTV Platform</b><br><font size='-1'>V${VERSION}</font>"
FOOTER="<font size='-1'>SageTV is a trademark or registered trademark of
 Google, Inc. in the US and other countries.<br>Copyright 2000-2012
 Google, Inc. 1600 Amphitheatre Pkwy<br>Mountain View, CA, 94043, U.S.A.
 All Rights Reserved.</font>"
WINDOWTITLE="SageTV V${VERSION} API Specification"


javadoc \
  -classpath temp-bin \
  -package \
  -windowtitle "${WINDOWTITLE}" \
  -doctitle "${DOCTITLE}" \
  -header "${HEADER}" \
  -bottom "${FOOTER}" \
  -d  ${OUTDIR} \
  -sourcepath temp-src \
  sage.api sage.media.rss


echo "copying to stvs/doc"
cp -r temp-doc/* ${BASEDIR}/stvs/doc

[jusjoken]

Originally Posted by Narflex

It took some digging around...but I finally found the script that I had which generated the javadocs from the code. You should be able to use this to generate the new PersonAPI. If you get this generated, you can just email me the zip file for it and I can then also upload it to the server so it's there as well.

#!/bin/bash

cd `dirname $0`
pwd


BASEDIR=../..
SRCDIR=${BASEDIR}/java
OUTDIR=temp-doc

if [ "${JDK_HOME}" ] ; then
  export PATH=${PATH}:${JDK_HOME}/bin
fi

for testprog in javac javadoc java ; do
  if [ -z `which $testprog` ] ; then
    echo $testprog not on search path - add JDK to PATH or run with JDK_HOME set
    exit 1;
  fi
done


rm -fr temp-doc temp-src temp-bin

# Compiling some of the required classes -- ignore any errors
mkdir temp-bin

# exit on error from here on...
set -e
javac -d temp-bin -sourcepath ${SRCDIR} \
   ${SRCDIR}/tv/sage/util/StudioAPIProcessor.java

echo ""
echo run API processor to build dummy java files from api source files
mkdir temp-src
java -cp temp-bin \
   tv.sage.util.StudioAPIProcessor \
   ${SRCDIR}/sage/api/ \
   temp-src/sage/api

mkdir -p temp-src/sage/media/rss
cp -r ${SRCDIR}/sage/media/rss temp-src/sage/media/

# remove ReflectionAPI as that is indended to be hidden
rm temp-src/sage/api/ReflectionAPI.java


echo ""
echo "Building Javadocs"

# get version num from source Version.java
eval $(grep -e '_VERSION *=' ${SRCDIR}/sage/Version.java |\
       sed -e 's/.* \(.*_VERSION\) *= *\(.*\);/\1=\2/')

VERSION=${MAJOR_VERSION}.${MINOR_VERSION}
DOCTITLE="SageTV<sup><font size='-2'>TM</font></sup> V${VERSION} API Specification"
HEADER="<b>SageTV Platform</b><br><font size='-1'>V${VERSION}</font>"
FOOTER="<font size='-1'>SageTV is a trademark or registered trademark of
 Google, Inc. in the US and other countries.<br>Copyright 2000-2012
 Google, Inc. 1600 Amphitheatre Pkwy<br>Mountain View, CA, 94043, U.S.A.
 All Rights Reserved.</font>"
WINDOWTITLE="SageTV V${VERSION} API Specification"


javadoc \
  -classpath temp-bin \
  -package \
  -windowtitle "${WINDOWTITLE}" \
  -doctitle "${DOCTITLE}" \
  -header "${HEADER}" \
  -bottom "${FOOTER}" \
  -d  ${OUTDIR} \
  -sourcepath temp-src \
  sage.api sage.media.rss


echo "copying to stvs/doc"
cp -r temp-doc/* ${BASEDIR}/stvs/doc

Thanks for this. I will take a shot at it and let you know what I come up with.

K

[jusjoken]

Attached is a test version which includes the PersonAPI.

Please test and let me know if this solves the issue for you and I will then release a new version. I have not yet looked at the other items you PM'd me with but will as time permits.

https://forums.sagetv.com/forums/att...1&d=1648993256

K

[jusjoken]

This is now fixed and released...

https://forums.sagetv.com/forums/sho...&postcount=422

K