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 scripting engine #116

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

NickPolyder
Copy link
Contributor

TBD

@PanosK92
Copy link
Owner

  • Issue with ScriptEngine Initialization: The initialization fails at step 2 due to the root_path being hardcoded.
  • Platform-Specific Code: While other platforms are not currently a focus, I'm glad to see that you conditionally include (ifdef) their potential code paths. This approach allows for easy future integration with the script engine on these platforms.
  • Integration with Entity Component System: The aim is to integrate this into the entity component system philosophy. This will involve creating a component that can be added to an entity. This component will store a script path, and the script will execute whenever the component is updated by the engine.

Your initiative in enabling C# scripting are appreciated, as it will allow those with higher-level programming skills to experiment with the engine. I'm here to assist further.

@NickPolyder
Copy link
Contributor Author

Hello @PanosK92 I have fixed the root path to be dynamic. Can you have a look and let me know if it is an acceptable solution ?

I would probably need your help with premake on including the CSharp SDK. The premake uses the old version of csproj and I am not familiar with it enough to find a suitable solution.

TODO:
-- Need to verify the linker libraries part
-- Need to create project specific settings (for where to load the dlls from etc)
-- Need to create Spartan.SDK (methods, structs) to allow for hooks and calls for every tick etc.
Amend build scripts to include the new perquisites
Basic .NET 7 Project and Engine Class.
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.

None yet

2 participants