Video Preview in EmulationStation

Nismo

@fieldofcows I finally succeeded in reproducing the issue of relative path!!! (yes still not working well on beta 7), and why some paths are changed and some paths not.

Ok let's start, i have only one game in my gamelist:

<?xml version="1.0"?>
<gameList>
  <game>
     <path>./Contra.zip</path>
     <name>Contra</name>
     <image>~/.emulationstation/downloaded_images/nes/Contra-image.png</image>
     <video>~/.emulationstation/Videos/Contra.mp4</video>
     <marquee>~/.emulationstation/Marquees/Contra.png</marquee>
  </game>
</gameList>

If ES find both videos and marquees the path changes but still relative and ok:

<image>~/.emulationstation/downloaded_images/nes/Contra-image.png</image>
<video>./Videos/Contra.mp4</video>
<marquee>./Marquees/Contra.png</marquee>

If ES doesn't find video but find marquee:

<image>~/.emulationstation/downloaded_images/nes/Contra-image.png</image>
<video>Z:/EMU//.emulationstation/roms/nes/Videos/Contra.mp4</video>
<marquee>./Marquees/Contra.png</marquee>

If ES find video but not marquee:

<image>~/.emulationstation/downloaded_images/nes/Contra-image.png</image>
<video>./Videos/Contra.mp4</video>
<marquee>Z:/EMU//.emulationstation/roms/nes/Marquees/Contra.png</marquee>

If doesn't find both of them:

<image>~/.emulationstation/downloaded_images/nes/Contra-image.png</image>
<video>Z:/EMU//.emulationstation/roms/nes/Videos/Contra.mp4</video>
<marquee>Z:/EMU//.emulationstation/roms/nes/Marquees/Contra.png</marquee>

So i hope this help you to find what's the problem. Yes still double back slash issue :(

Regards.

tronkyfran

@Nismo I have a theme on a half way since some time, and its very similar to yours in concept, hope you dont mind. In any case it will take a long time to finish because its 3d and all that, so maybe youll forget about this!! xD

Nismo

@tronkyfran Of course no problem mate, when more themes coming for this fork, will be better.

And the theme is not only my theme, like i said before it's a port from the original oldroom theme for attract mode: http://forum.attractmode.org/index.php?topic=926.0

Credits to Spinelli (psd file), Cobra83 and bundangdon for some backgrounds, credits to litzoid for system art. (All is in the readme file included in the theme)

I put a lot of work in this theme, i pick some backgrounds from there (of course under permission of all creators and litzoid too) and i added a lot of things that not there, reflections, more stuff, figures, cartridges, etc.

And placing all the image files and videos is a pain, he he he, and edit a lot of xml files, you know.

The theme originally only have a few systems, about 4 or 5 , the rest it's original from me (i have 14 systems now), and i share my work there to contribute the project.

Anyway is your theme compatible with this version?, my theme doesn't work on "vanilla" version of emulationstation.

Good look with your theme, i whould like to see some screens... XD

Regards.

fieldofcows

@Nismo said in Video Preview in EmulationStation:

my theme doesn't work on "vanilla" version of emulationstation.

I'm working on that :) I have spent some time splitting just the video view out into a separate branch. When that is done and tested I'll submit a pull request to the retropie EmulationStation fork and see what happens.

Don't worry @Nismo, I'll maintain a development branch with all the new bits we've been working on and not just the video for your Windows platform.

Rion

@fieldofcows Just because I think you're doing such a great job and it's Friday im sending you a virtual Beer!

fieldofcows

@Rion said in Video Preview in EmulationStation:

@fieldofcows Just because I think you're doing such a great job and it's Friday im sending you a virtual Beer!

Haha! Received with thanks. I'll drink it later - after work :D

fieldofcows

@Nismo said in Video Preview in EmulationStation:

I finally succeeded in reproducing the issue of relative path!!!

I've now reproduced it to thanks to your info. This is a pre-existing issue in ES and not related to the video view changes. I think my experimental metadata patch should fix this. I'll put together a fix and pull it into my Windows dev branch and give you a new build when it's done.

Nismo

@fieldofcows Thank you, finally you have some advances to fix it. What about the double back slash? some advances?

Like i said before you're fixing a few old issues of ES (like the control pad L and R on gamelist). Nice work.

BuZz

@Zigurana said in Video Preview in EmulationStation:

But I agree with you both, delivering back to the Retropie branch should be the end goal. Its reassuring to see Buzz come around to that point of view.

I don't think my view has really changed. I have been accepting PR for our ES for over a year. I still won't accept anything ofcourse - it depends what it is, how it is written/tested, and whether it is maintainable. (eg to avoid issues like we had after the directLaunch functionality).

To add to the discussion above about feature branches etc - It would be one PR per added "functionality". I won't accept a PR with 20 new features and 10,000 lines of code changes that are impossible to review :)

Bluescreen

is there an sd image with video preview for download?
i can´t compile by myself ;(

Nismo

@Bluescreen not yet, but fieldofcows it's working to provide a build for who want to test.

tronkyfran

@Nismo
Well, its work in progress right now, Just have some pictures of different
perspectives of the room. I have lots of work to do yet, custom videos,
images, etc...you know. The main problem is that I have to convert every
video preview to my 3d perspective and its really time consuming.But in
exchange I think it looks ok. Im not ready to show something yet but Ill be in
two or three months, dont want to do the same that I did with the
Background Images and get an incomplete set, so Ill wait till aI have snes
with at least 100 videos converted or so and then you can give some
feedback ;)
Keep on your amazing work!!!!

Nismo

@tronkyfran can't wait to see your theme... XD.

Nismo

@fieldofcows what's your next step?

capsaicin

@Bluescreen said in Video Preview in EmulationStation:

is there an sd image with video preview for download?
i can´t compile by myself ;(

SSH to Pi and execute the following commands:

# install packages
sudo apt-get update
sudo apt-get --yes install vim vlc libvlc-dev libvlc5

# get and compile
git clone https://github.com/fieldofcows/EmulationStation.git
cd EmulationStation/
cmake . -DFREETYPE_INCLUDE_DIRS=/usr/include/freetype2/
make clean
make

# auto start video emulatorstation
sudo sed -i 's/\/.*emulationstation\.sh/\/home\/pi\/EmulationStation\/emulationstation\.sh/' /usr/bin/emulationstation

fieldofcows

@Nismo said in Video Preview in EmulationStation:

@fieldofcows what's your next step?

Well, I've spent whatever free time I've had sorting out my repository and splitting each 'feature' into a separate branch. That process isn't quite complete yet. I am going to submit some of the changes as PR's to the RetroPie fork and see what happens. I haven't had any feedback from anyone trying out the Pi build so maybe there's not that much interest - or everyone is waiting until it is in an 'official' build.

Once I've done that I'll get back to developing some more features and fixing some bugs.

Do you have a Windows version wishlist? I assume metadata fixes would be on top :)

Nismo

@fieldofcows yes i've posted some wishes post ago... but i'll let you concentrate in your splitting process.

I was painting my house so not much time to work on theme.

I've finished gb theme and working on gbc theme, but i'll upload new theme version when more systems added.

Bluescreen

@capsaicin Thx it works :)

Nismo

New little update for oldroom theme:

added Game boy system.
added Game boy Color system. (I need better gbc image for this theme).
Fixed detailed view.

Somebody have a Game boy Color and can make a photo holding it in his hand?

Now detailed view looks this way :

alt text

Download: http://www.mediafire.com/file/ur2k9sbp2ua6yn9/oldroom_1.7_720p.zip (Recommended for Rpi)

1080p version: http://www.mediafire.com/file/vvylrsgh8waqkkk/oldroom_1.7_1080p.zip

Sorry @fieldofcows this time i didn't have time to provide an 800x600 fix but i'll do soon.

I can't make work marquee on top of "rating" on detailed view, i'm not sure if can't be possible on detailed view or it's my fault.

Enjoy.

Rion

@Nismo @fieldofcows How hard would this be to implement for other themes for example to Pixels by @Rookervik ?

I mean you don't have to change much just an option to replace the picture with an video if that exists and if not just show the screenshot/boxart?

Or are there more underlying changes that have to be made?