Ongoing Development Thread for 1.0.0 [now closed] (1 Viewer)

[seven5]

heres soemthing else. for soem reason its not adding shows from this: http://thetvdb.com/?tab=series&id=80140&lid=7


I added that to the THETVDB so the plugin would pick it up. The regex's are getting the show, so says the test. But for soem reason its not adding that show for me.

Doing a search for that show in the xml interface for tvdb brings back a TONNNNNNNN of results, maybe its timing out? I'd think that getsereies.php should require the exact name, instead it seems to be searching on "the", "totally", "rad", "show". which obiviously will bring back a bazillion results.....

 

[Inker]

I finally installed the newest SVN, and I have to say, I really don't like the episode image merged with the tv show icons. It just doesn't seem to be within the same scope.

The icons are 'describing' the episode's quality and format. Not content. And it looks kinda crappy in my skin. Is there a way to overide it?

The skin authors can seperate the two by adding a skin control for the episode images. If that is not found, it will be appended to the logos. Spinsafe Revolve does this for instance.

when i delete something, it asks me to confirm twice. Thats seems redundant. It seems that the user should be responsible for knowing what their setting is in the configuration. Having it NOT delete by default should be enough. Its a bit obnoxious. The scroll is a bit ghetto too when its tring to keep the "would you like to proceed" on the screen while the rest scrolls off. I say just dump the second confirmation completely, 4 presses/clicks is a bit much when i just want to delete something.

True, but I thought it was useful so you could choose between deleting the physical files and just deleting db entries. Maybe I'll remove it.

When in ALL mode and scrolling through the wide icons, sometimes its snappy, and sometimes its not. Its like its generating something. But i'll go up and down over and over. so anything thats getting cached should have stayed cached. I'm not sure why its sometimes lagging and sometimes snappy.

Hmm, I don't know, for me its very snappy. Might have something to do with Video memory getting swapped when you have tons of shows (all those banners).

Going to LATEST is a lot faster now, is that because the range of time is shorter so its less shows?

Its because the logos-engine has been optimized. I don't know what you mean by "the range of time is shorter now".

heres soemthing else. for soem reason its not adding shows from this: http://thetvdb.com/?tab=series&id=80140&lid=7


I added that to the THETVDB so the plugin would pick it up. The regex's are getting the show, so says the test. But for soem reason its not adding that show for me.

Doing a search for that show in the xml interface for tvdb brings back a TONNNNNNNN of results, maybe its timing out? I'd think that getsereies.php should require the exact name, instead it seems to be searching on "the", "totally", "rad", "show". which obiviously will bring back a bazillion results.....

It shouldn't time out, might be an issue with the very latest svn from yesterday where something is wrong with the sql (don't know what yet). But I was getting some weird issues after a while last night when testing.

 

[seven5]

The skin authors can seperate the two by adding a skin control for the episode images. If that is not found, it will be appended to the logos. Spinsafe Revolve does this for instance.

so by NOT putting a control, the thumb is shown? That seems kind of backwards. Why would we explicitly put a control there, when we dont want it there? so would we have to add the control and then give it a negative X coordinate? That seems kinda funky.


As for the range, i meant the range in days, its pulling "RECENT SHOWS" from. it seems to only be a week now, rather than 2 weeks.

As for being more snappy, i pumped MP's priority up to max, and that helped a ton. Everything is a bit more snappy now. Dont know how i missed that before. It was set for default, which is wrong, seeing that this is a dedicated mediacenter.

Thanks Inker!

 

[Inker]

The skin authors can seperate the two by adding a skin control for the episode images. If that is not found, it will be appended to the logos. Spinsafe Revolve does this for instance.

so by NOT putting a control, the thumb is shown? That seems kind of backwards. Why would we explicitly put a control there, when we dont want it there? so would we have to add the control and then give it a negative X coordinate? That seems kinda funky.

Episode images are part of the "deal", they are shown wheter you put a special control for it or not. You can either have them shown where logos are shown, which requires no skin update, or you can show them anywhere by putting a special control for it in your skin. Why would not want them to be shown?

As for the range, i meant the range in days, its pulling "RECENT SHOWS" from. it seems to only be a week now, rather than 2 weeks.

As for being more snappy, i pumped MP's priority up to max, and that helped a ton. Everything is a bit more snappy now. Dont know how i missed that before. It was set for default, which is wrong, seeing that this is a dedicated mediacenter.

Thanks Inker!

Latest view has nothing todo with how many days, it displays the last (I think) 25 episodes ordered by airdate. If you have 25 shows within a week, then the range in days will be smaller but the amount of work remains exactly the same.

I don't know why it wasn't snappy for you, it does have to do a bit of work (which come to think of it is easy to reduce, so I think performance can be improved) when selecting a new item, but that work should be done in ms normally, certainly not noticable (which it isn't for me).

 

[pannivas]

yes i find the episode thumbs a really cool feature.

 

[seven5]

It was snappy after i boosted MP's priority. Its perfectly fine now.

I do understand the value of the episode thumbnails. But what i dont understand is the assumptions you're making for the skin designers. A thumbnail for the episode is not within the same context of the icons. Icons describe format and quality. The episode thumb does not, it describes content.

If it has its own control for it, let the skin designer use that. It seems a bit redundant to have it in two places anyways, for your code. Like i said, if somoene does not want the thumbnail, how does he get rid of it? Add the control and push it off the screen with a negative coordinate? That should be a sign of a problem with implementation. I think the common convention with gui design is, if you dont want a control, you dont add it. Why is that broken here?

I'm all for the thumbnail, i think its great. I love having it in the full screen OSD, but just pushing it into the episode icon list is wrong. And its really a bit worthless, seeing that it ends up the same size as the icons, which isn't very easy to make out.

View attachment 7007

 

[Inker]

Seriously seven, it was done so not yet another skin update is needed and the thumbnail works without users skin/author needing to update. Thats the way it is and that's the way it will stay. Bad luck if you want to get rid of it.

I don't want to argue over the most minuscule details when there are so many more important things at hand.

 

[seven5]

Bad luck if you want to get rid of it.

well thats a crappy attitude.

I don't want to argue over the most minuscule details when there are so many more important things at hand.

I'm not arguing, I was discussin. Sorry if it seems that way. I just appreciate the plugin enough that I want to help make it as good as it can.

 

[karamellz]

Well it's just easier if you talk to the person who designs the skin you use. Inker handles what happens "behind the screen", and makes it as versitile as possible for the skin designers. It's up to them to make it sexy. Just look at what mirk has done with the Spinsafe Revolve. Smokin' hot!



Karamellz~

 

[seven5]

yah, i posted about it at the ReVision dev thread. I'm gonna hack on the xml file tonight to play with it.