search

thorbenk / blockedarray

read/write to a blocked, in-memory compressed array
Other
8 stars 2 forks source link

Double registration of TinyVector converters #2

Open burgerdev opened 10 years ago

burgerdev commented 10 years ago

The ExportV.export_() call in blockwisecc_py.cxx, at line 106, implicitly registers a to-python converter for vigra::TinyVector, which is probably already registered in vigra. Should be replaced by an explicit from-python converter.

Warning:

$ python -c 'import blockedarray'
/home/markus/Coding/hci/blockedarray/blockedarray/__init__.py:1: RuntimeWarning: to-Python converter for vigra::TinyVector<int, 2> already registered; second conversion method ignored.
  from _blockedarray import *
/home/markus/Coding/hci/blockedarray/blockedarray/__init__.py:1: RuntimeWarning: to-Python converter for vigra::TinyVector<int, 3> already registered; second conversion method ignored.
  from _blockedarray import *
thorbenk commented 10 years ago

Would you like to prepare a pull request for this?

burgerdev commented 10 years ago

On 08.02.2014 08:21, thorbenk wrote:

Would you like to prepare a pull request for this?

Reply to this email directly or view it on GitHub: https://github.com/thorbenk/blockedarray/issues/2#issuecomment-34537615

I would like to, as soon as I figure out how to do it ;)

burgerdev commented 10 years ago

Seems like the converters are not necessary, vigra registers converters for vigra::TinyVector<N,MultiArrayIndex> to python tuples by default. But boost still complains about missing converters when properties are exposed, methods work fine.