fixstars / ion-kit

Modernized graph-based data processing framework
MIT License
7 stars 6 forks source link

with_bb_module not work #223

Closed Fixstars-momoko closed 7 months ago

Fixstars-momoko commented 8 months ago

with ionpy 1.2.0, the line to load dynamic module caused the error. The program works without the following line.

builder.with_bb_module('ion-bb') 
xinyuli1204 commented 8 months ago

I don't know if this is the reason, but inside ionpy, it has prebuilt dll which based on ion-kit 1.2, but in sensing dev it seems to have 1.0., which is not compatible. Maybe we can change native.py , and make it rely on sensing-dev tentatively.

if platform.system()  == 'Windows':
    ion_core_module = find_library('ion-core.dll')
xinyuli1204 commented 8 months ago
  1. This happens only on windows
  2. Currently only on windows, ion-core.dll is not loaded from /module/ion-core.dll
xinyuli1204 commented 7 months ago

Done https://github.com/fixstars/ion-kit/pull/247