BYU-PRISM / Seeq

Hybrid (Physics-Informed) Machine Learning
MIT License
46 stars 5 forks source link

SysID add-on MV and CV selection does not appear to work with Seeq R57. Works fine with R56 #30

Closed anvenkat18 closed 1 year ago

anvenkat18 commented 2 years ago

The add-on works fine with R56 but fails to load/show MV and CV signals in R57 (shows no data available). I know we had seen that earlier and attempted to address it. @imanbabaei let's taker a look once you are in. While we are at it, lets check R58 also.

anvenkat18 commented 2 years ago

R56:

R56

R57:

R57
imanbabaei commented 2 years ago

It looks like starting from seeq==56.1.8.184.12, we have some changes in spy.pull and spy.search: spy < 56.1.8.184.12: spy.pull(... , quiet=True)

spy >= 56.1.8.184.12: spy.pull(... , status=spy.Status(quiet=True))

Let's release v0.4.1 apply this change and the fix for issue #22, and check again.