-
Notifications
You must be signed in to change notification settings - Fork 113
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
Upload collection metadata #117
Comments
@agneevX please explain what do you mean. scrobble in my dictionary means to use "user is watching x" movie play scrobbling is possible with tv episode scrobbling does not work yet |
metadata = picture definition (720p, 1080p,...), audio channels and codec, etc... |
ok, seems it's about this feature: but can't find any method from trakt library we are using to support this. is there public api for this? |
Sorry, I misunderstood the definition of "scrobble" 😅 |
Worth noting is that the Plex-trakt-scrobbler doesn't support This info can perhaps be retrieved from the file name and analyzed metadata perhaps. |
adding missing link: @agneevX please link to resources you are talking about! |
support from trakt library we are using is missing: |
I was able to make up these fields: def to_json(self):
return {
"collected_at": timestamp(self.collected_at),
"media_type": "digital",
"resolution": self.resolution,
# "hdr": "dolby_vision",
# "audio": "dts_ma",
"audio_channels": self.audio_channels,
} |
for audio channels, need exact values from plex side for these:
trakt was so picky, that it gave 500 error if for audio channels was submitted the same for videos, need plex side values for:
And audio format:
dirty code is here now: #211, maybe moved out soon, as that pr is for |
so, @agneevX (and others), if you want to progress this faster, you can help by finding values to use for re-mapping. Maybe there's simpler, easier to find, but there's XML files in:
they contain lines like: <Stream index="1" selected="1" streamType="2" channels="2" codec="aac" languageCode="en" id="2"/> |
Changes for only this issue are here now: |
@glensc I'm not exactly sure what it is you want me to do... do you want me to provide stream metadata info for media in my library? |
If so, I hope these help.
|
@agneevX is there info on how you obtained those XML's? for me and for posterity. I grepped through mine, they all were channels=2, which is weird. |
@agneevX this is very confusing to me, can you put up a table or logic on how to map these together for trakt?
for audio it seems simple, take but video, it has tons of attributes: this area is very foreign here, I'm just trying to map available data, so help is appreachiated. for example are I've added
|
I've now added a test case, so could mock the objects and see if the expected result is built:
|
Ok, so these are different.
HLG is a HDR format, more on that below. The other one is the text metadata label, it should be ignored. This is the logic I'd say to determine HDR format: Dolby Vision check - if else check for HLG - if else check for HDR/HDR10 - if else HDR is not present. (I am not sure about |
Since there's no interest from anyone to write code for this, I'm going to merge existing PR as it is: |
Maybe add a |
added label. would you update readme to add a section so people actually find such issues? |
Added doc how to share xml: |
re-opening, as the current code submits typical 720p movies as 576p: the thread there discusses a possible solution. |
It would be nice if metadata support for collections were added.
The text was updated successfully, but these errors were encountered: