Baunsgaard commented on PR #2282: URL: https://github.com/apache/systemds/pull/2282#issuecomment-3018127571
Hi @Paulo-Mansano , Thanks for opening the PR! It seems we slightly miscommunicated, since the intention is to 'modify' the existing transformations to construct ragged arrays as their metadata, not to create new transformations. In specific we need to look at : https://github.com/apache/systemds/blob/692dcc3d1c46c79b281bfc85686d750d62f427ad/src/main/java/org/apache/sysds/runtime/transform/encode/Encoder.java#L70 Where the metadata is allocated, (we want to modify it to allocate the ragged array) and https://github.com/apache/systemds/blob/692dcc3d1c46c79b281bfc85686d750d62f427ad/src/main/java/org/apache/sysds/runtime/transform/encode/Encoder.java#L63 Where the metadata frame is populated with information. (we want it to be in the form of a ragged array) I hope this helps clarify things. -- 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: dev-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
