Skip to content

Conversation

@JaydenzKoci
Copy link

Adds basic features for video backgrounds. You have to have a file called video.mp4 in the song folder for it to detect at the moment. I'll be adding video_start_time as well later on. Also added Hud Position Setting for the older score display "don't know if we still using it or not but yea"

@JaydenzKoci
Copy link
Author

Also for the time being the ffmpeg files being built already in the directories is probably the smarter way to go with it until we just make a repo for ffmpeg at some point

@solamint
Copy link
Collaborator

solamint commented Oct 2, 2025

Also for the time being the ffmpeg files being built already in the directories is probably the smarter way to go with it until we just make a repo for ffmpeg at some point

I would rather use CMake's importing features to automatically download the ffmpeg files so we don't have to deal with a 155k diff.

@JaydenzKoci
Copy link
Author

That's going to be the annoying part. Last time I built it. It took like 3-5 mins 💀

@JaydenzKoci
Copy link
Author

If you want I can go make a repo with the built files and we can try to go from there

@JaydenzKoci
Copy link
Author

Update: I'll be trying to get ffmpeg to build and cache along side the build process instead of having all the files in the lib folder

@JaydenzKoci
Copy link
Author

I think it's good now lol

the lib we use for discord does not support linux the way its used at the moment. I basically just made a function that makes the game load without the discord lib so it can load lol
@JaydenzKoci
Copy link
Author

I'll likely finish the pr on Saturday or Sunday.

}
#endif

#if FFMPEG_AVAILABLE
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VideoStream's stubbing should take place in video.cpp, not video.h.

// Created by Jaydenz on 9/4/2025.
//

#include "video.h" No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementation details for VideoStream should be in the source file, not the header file.


#include "settings.h"

namespace Encore::RhythmEngine {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this in Encore::RhythmEngine?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants