Ruby wrapper about paratext HOT 5 OPEN

Hi.

I went with SWIG because it does a lot of the hard work of parsing and validating input for you. If you write your own wrapper, you have to write a lot of boilerplated code yourself. Given a large enough codebase (100K+ lines), custom wrappers become very hard to maintain.

The Python typemap lives here: https://github.com/wiseio/paratext/blob/master/src/python. The interface file (python.i) is pretty rudimentary. It converts std::vector<int/double/string> to NumPy vectors.

%typemap(out) std::vector<int> {
  $result = (PyObject*)::build_array<std::vector<int>>($1);
}

%typemap(out) std::vector<double> {
  $result = (PyObject*)::build_array<std::vector<double>>($1);
}

%typemap(out) std::vector<size_t> {
  $result = (PyObject*)::build_array<std::vector<size_t>>($1);
}

%typemap(out) const std::vector<std::string> & {
  { auto range = std::make_pair($1->begin(), $1->end());
   $result = (PyObject*)::build_array_from_range(range);
  }
}

%typemap(out) std::vector<std::string> {
  $result = (PyObject*)::build_array<std::vector<std::string>>($1);
}

%typemap(out) ParaText::CSV::ColBasedPopulator {
  $result = (PyObject*)::build_populator<ParaText::CSV::ColBasedPopulator>($1);
}

You need to write template functions to convert from C++ containers to Ruby arrays.

Ruby::Value build_array<ContainerType>(const ContainerType &container)

You also need to write a function to convert CSV columns to Ruby arrays:

Ruby::Value build_populator<ParaText::CSV::ColBasedPopulator>(ParaText::CSV::ColBasedPopulator &populator)

This is needed to call the loader.get_column(column_index) function.

Best,
Damian

from paratext.

I am not very familiar with the internals of Ruby, but I am happy to provide guidance for the effort.

from paratext.

Works :)

Should we use SWIG or is it ok to use the Ruby C API directly? Any specific reasons why you used SWIG for the python bindings?

from paratext.

SWIG will likely make your life easier if you have straightforward type mappings. I may write a Cython binding for paratext (mainly to simplify talking to other Python modules that use Cython -- you can define an internal C API via pxd files) but either way it makes for less work than a hand-coded C extension for Python at least.

from paratext.

Alright. I'll explore all the options and post them here soon. Thanks :)

from paratext.