janniklinde commented on PR #2560:
URL: https://github.com/apache/systemds/pull/2560#issuecomment-5166813297

   Generally, all functions that are expected to provide a working 
implementation should be tested, @Mancer1. Coverage also includes nested 
function calls, so you should be able to achieve high coverage by testing the 
publicly available functions of the compressed column group. Please use the 
tests for existing `AColGroup` implementations as a reference.
   
   Also, please avoid reimplementing helper functionality that is already 
available in the test utilities, such as random matrix generation.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to