Skip to content
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

Feature: Add a project launcher #67

Open
omeryusufyagci opened this issue Nov 8, 2024 · 5 comments · May be fixed by #75
Open

Feature: Add a project launcher #67

omeryusufyagci opened this issue Nov 8, 2024 · 5 comments · May be fixed by #75
Assignees
Labels
feature New feature or request help wanted Extra attention is needed platform urgent

Comments

@omeryusufyagci
Copy link
Owner

omeryusufyagci commented Nov 8, 2024

This will manage the platform specific configuration/installation of dependencies. It should automatically update config.json file so that we can avoid adding a separate field for each platform and dependency that the user has to verify and maintain.

This will also introduce more flexibility in how to start our applications built on top of MediaProcessor.
The launcher should not assume the current web-based application will be the only use case. In the future we might provide a desktop GUI, and potentially other standalone applications.

The goal is to achieve a cross-platform seamless user experience.

@omeryusufyagci omeryusufyagci added feature New feature or request help wanted Extra attention is needed platform urgent labels Nov 8, 2024
@nikhiljangra264
Copy link
Contributor

Could you please elaborate it more

@omeryusufyagci
Copy link
Owner Author

Hey @nikhiljangra264, thanks for the question!

What we're aiming for here is to :

  • Provide a better first-time experience by finding/installing dependencies,
  • Transform the config.json into a dynamic config file that will be updated wrt platform,
  • Ensure to keep the current ease of use while being easily extensible in the future for other applications.

I was thinking of a script (or a few small scripts) that would try to locate the dependencies and install missing ones, update the config.json file accordingly and provide means to start applications built on top of MediaProcessor. Currently we only have the Web app, but it should be extensible; it could just take args, e.g. --web or --gui, for now and that would be OK.

We should of course handle C++ specific dependencies within the existing CMake files as much as we can, and we could introduce other tools where they make sense, but I hope this gives a better view of what I'm looking for here.

Does this help? Please let me know in case of any questions/feedback!

@nikhiljangra264
Copy link
Contributor

we could add two scripts like .sh for linux and mac and .bat for windows

@nikhiljangra264
Copy link
Contributor

I will do this.
Please assign me.

@omeryusufyagci
Copy link
Owner Author

we could add two scripts like .sh for linux and mac and .bat for windows

Or we could go with python, it's already a project dependency. So, it's not like it'd cost us anything.

Please assign me.

All yours ! : ))

omeryusufyagci added a commit that referenced this issue Nov 12, 2024
As part of #67 the configurations is planned to be dynamically loaded.
This is a hacky test to see if we can use this in the meantime.
@nikhiljangra264 nikhiljangra264 linked a pull request Nov 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed platform urgent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants