CGiachalis

Results 22 comments of CGiachalis

Indeed, `katex` option didn't work either out of the box. Thanks for confirming @nanxstats .

As a last note, it seems at C++ level the group getter is assigned 'key' attribute whereas 'names' for array although the code logic is identical. *libtiledb_array_get_metadata_from_index* https://github.com/TileDB-Inc/TileDB-R/blob/c2ba622f7ca0e5bb448f127e0e113bcac277a486/src/libtiledb.cpp#L2878-L2879 *libtiledb_group_get_metadata_from_index* https://github.com/TileDB-Inc/TileDB-R/blob/c2ba622f7ca0e5bb448f127e0e113bcac277a486/src/libtiledb.cpp#L5434-L5435

Thanks for your response @rcurtin! Indeed, we need to add the class definition towards the end of `src/mlpack/bindings/R/print_R.cpp`. With respect to native serialisation routines, it's better to tackle it separately...

> > So, I suppose a slight change to the print class right after: > > [mlpack/src/mlpack/bindings/R/print_serialize_util.hpp](https://github.com/mlpack/mlpack/blob/822c499f5d71fd9c401d45e0b9977d2aca02f7a0/src/mlpack/bindings/R/print_serialize_util.hpp#L64-L65) > > Lines 64 to 65 in [822c499](/mlpack/mlpack/commit/822c499f5d71fd9c401d45e0b9977d2aca02f7a0) > > MLPACK_COUT_STREAM > MLPACK_COUT_STREAM...

Right, I managed to get it done i.e., printing `class(out)

Basically, I created a replicate of `print_serialize_util.h` -> `print_serialize_util2.h` and modified one of the template functions to get the desire output so I can use it in `print_R.cpp` ; Now...

FYI ```c++ /** * For non-matrix serializable types we need to print something. */ template void PrintSerializeUtil2( util::ParamData& d, const typename std::enable_if::value>::type* = 0, const typename std::enable_if::type* = 0) {...

> Hm. But why no S3 dispatch when your very Subject: to this issue suggests to use a class (and hence R dispatch) ? Let's recap to ensure we are...

Here's the branch https://github.com/cgiachalis/mlpack/tree/3722-r-add-class