-
Notifications
You must be signed in to change notification settings - Fork 2
Add RSA blocks #43
base: main
Are you sure you want to change the base?
Add RSA blocks #43
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ps://github.com/flojoy-ai/blocks into dallas/stu-196-add-rsa-api-and-fix-gallery-apps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main concern is the dll that's loaded in each block, hardcoded path and windows only seems a bit weird to me
defau = "C:/Tektronix/RSA_API/lib/x64/RSA_API.dll" | ||
moved = "C:/Program Files/Tek/RSA_API/lib/x64/RSA_API.dll" | ||
if path.isfile(defau): | ||
rsa = cdll.LoadLibrary(defau) | ||
elif path.isfile(moved): | ||
rsa = cdll.LoadLibrary(moved) | ||
else: | ||
raise FileNotFoundError( | ||
"Cannot find RSA_API.dll. Download from: https://www.tek.com/en/products/spectrum-analyzers/rsa500" | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this dll always located here? Also, this only works on windows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's only available for Windows. The default install location is the second one but I moved mine because I didn't want it on C: directly.
Studio PR: flojoy-ai/studio#987