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

Update LLama.Examples using Spectre.Console #255

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

xbotter
Copy link
Collaborator

@xbotter xbotter commented Nov 6, 2023

  • Refactored TestRunner.cs to use a dictionary to store the available examples and prompt the user to choose one to run.
  • Added a new feature to TestRunner.cs to display a rule before executing an example.
  • Improved the user interface of TestRunner.cs by clearing the console after each iteration.

- Handle invalid choice in Examples.TryGetValue
- Add exit condition for choice "Exit"
- Display a rule before executing example
- Clear console after each iteration

📝 This change refactors the TestRunner.cs file. It handles invalid choices in Examples.TryGetValue, adds an exit condition for the choice "Exit", displays a rule before executing an example, and clears the console after each iteration.
@martindevans
Copy link
Member

I haven't seen Specte.Console before, it looks really good!

@martindevans martindevans merged commit d993beb into SciSharp:master Nov 7, 2023
3 of 4 checks passed
@martindevans
Copy link
Member

@xbotter unfortunately an accident with git revert in another PR managed to remove this work. Would you mind resubmitting this PR? Sorry about that!

@xbotter
Copy link
Collaborator Author

xbotter commented Nov 15, 2023

@xbotter unfortunately an accident with git revert in another PR managed to remove this work. Would you mind resubmitting this PR? Sorry about that!

Take it easy. I will make some adjustments and submit them together.

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.

2 participants