search

src-d / ml-backlog

Issues belonging to source{d}'s Machine Learning team which cannot be related to a specific repository.
0 stars 3 forks source link

Test bblfsh client v3 #61

Closed vmarkovtsev closed 5 years ago

vmarkovtsev commented 5 years ago

We expect to have some difficulties while using new bblfsh client so it is better to test it beforehand

vmarkovtsev commented 5 years ago

We tested it. We found two major problems.

  1. Leaks, see https://github.com/bblfsh/python-client/issues/167
  2. Incomplete compatibility layer, fixed in https://github.com/bblfsh/python-client/pull/172

Resolution: once the leaks are fixed, we will be able to switch to UASTv2. src-d/ml-core has already been updated in https://github.com/src-d/ml-core/pull/33