[GitHub] [ignite] avplatonov commented on a change in pull request #6378: IGNITE-11647: ML Vectors should work with all Serializable objects besides double

[GitBox <gi...@apache.org>]

avplatonov commented on a change in pull request #6378: IGNITE-11647: ML Vectors should work with all Serializable objects besides double
URL: https://github.com/apache/ignite/pull/6378#discussion_r270659121
 
 

 ##########
 File path: modules/ml/src/main/java/org/apache/ignite/ml/math/primitives/vector/AbstractVector.java
 ##########
 @@ -160,6 +183,18 @@ protected void checkIndex(int idx) {
         return storageGet(idx);
     }
 
+    /** {@inheritDoc} */
+    @Override public <T extends Serializable> T getRaw(int idx) {
+        checkIndex(idx);
+
+        return sto.getRaw(idx);
+    }
+
+    /** {@inheritDoc} */
+    @Override public <T extends Serializable> T getRawX(int idx) {
 
 Review comment:
   Please read javadoc for get*X methods:
   "Gets the value at specified index without checking for index boundaries"

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services