-
-
Notifications
You must be signed in to change notification settings - Fork 410
Evaluator API Commands Example doc #5204
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
base: main
Are you sure you want to change the base?
Conversation
This isn't really a good example showing off the Evaluator API. There is too much code unrelated to Evaluator and it doesn't explain or highlight what is interesting about the APIs that Evaluator exposes to users |
Thank you for the feedback. Is it this style of writing the example doc for Evaluator API as is in mill built-in command here you was expecting? How I understand the issue description was to use the mentioned Evaluator commands to do something and then explain what each commands do in the sample project. |
The basic requirement is to clearly explain what the different APIs on the Evaluator object are useful for and how to use them. How you do so doesn't really matter, but the current PR does not successfully achieve that goal |
71f539b
to
685a361
Compare
What do you think of the updated doc? |
Looking through the examples, they are not sufficiently motivated to serve as useful explanations or examples to users. I think. And the |
1d3b959
to
3ba698a
Compare
13ae7cf
to
0c87e59
Compare
I made some updates
I can make more iterations if they're fit to be in the documentation |
To close #4598