NaXzyu / Dialogos

Dialogos: Pioneering Interactive Narratives and Language Proficiency with Enhanced AI in Unity
MIT License
3 stars 1 forks source link

Ensure `Doctor.bat` Verifies `ml-agents` Installation #3

Open p3nGu1nZz opened 5 months ago

p3nGu1nZz commented 5 months ago

Description

The Doctor.bat script currently does not check if ml-agents is installed on the system. This can lead to runtime errors for users who haven't installed ml-agents. To improve the user experience and catch this issue early, we should include a check for ml-agents installation within the Doctor.bat script.

Expected Behavior

When running Doctor.bat, the script should:

Current Behavior

The script executes without verifying the installation of ml-agents, which is a prerequisite for our project.

Possible Solution

Implement a check within Doctor.bat that looks for ml-agents in the system's PATH or in the expected installation directories. If ml-agents is not found, the script should inform the user and provide installation instructions.

Steps to Reproduce

  1. Run Doctor.bat on a system without ml-agents installed.
  2. Observe that there is no check or message regarding ml-agents.

Additional Context

This check is crucial for users who are setting up the project for the first time and might not be aware of all the dependencies.


Feel free to adjust the content as needed before submitting it to your project's GitHub repository.