Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PyCdec lattice edge weights/features broken #67

Open
armatthews opened this issue Feb 11, 2015 · 1 comment
Open

PyCdec lattice edge weights/features broken #67

armatthews opened this issue Feb 11, 2015 · 1 comment

Comments

@armatthews
Copy link
Collaborator

We updated cdec to allow a sparse vector of features rather than a single cost on each edge. PyCdec relies on storing edges in tuples, and while a double was fine in a tuple, a FastSparseVector is a C++ object and thus cannot go in a tuple, which is a python object.

The solution is probably to use a dictionary in the tuple, then have a function that converts a python dict to a FastSparseVector before throwing it back over the fence to the C++ side.

@ghost
Copy link

ghost commented Aug 13, 2015

This renders the whole Python module broken. It is not possible to build a realtime system as described by [http://www.cs.cmu.edu/~mdenkows/cdec-realtime.html], as the module won't build and thus not install.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant