-
Notifications
You must be signed in to change notification settings - Fork 224
Conversation
updated documentation, added separate sections for the data schema, the jar, and the ml_ops.sh script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the structure and the new content.
There are still a few things missing like the ones mentioned in SPOT-163 https://issues.apache.org/jira/browse/SPOT-163 and all the stuff in spot.conf regarding spot-ml.
I'm not saying this PR should resolve those, I'm just complementing the list.
Thanks.
I'll add some material to address SPOT-163 but what we probably really need somewhere is a "hey to configure your /etc/spot.conf" document |
added documentation on the parameters that ml_ops.sh grabs from /etc/spot.conf
Added documentation for how ml_ops.sh used the TOL parameter from /etc/spot.conf as a default value for the suspicion threshold.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
+1 |
1 similar comment
+1 |
|
||
The remaining instructions in this README file treat spot-ml in a stand-alone fashion that might be helpful for customizing and troubleshooting the | ||
component. | ||
These routines are contained in a jar file and there is a shell script ml_ops.sh for a simplified the invocation of the analyses. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems to be a typo here on new line 10, "...for a simplified the invocation..."
+1 (one type pointed out) |
'typo' that is |
updated documentation, added separate sections for the data schema, the jar, and the ml_ops.sh script