HAL-Webinar icon indicating copy to clipboard operation
HAL-Webinar copied to clipboard

Opened project in LV2019 and fixed errors.

Open jhgee opened this issue 5 years ago • 4 comments

Especially the following errors had to be fixed:

  • getter and setter didn't work anymore. The error was within the "In Place Element Structure" where the unbundle by name wire had to be reconnected which solved the problem
  • In several Process.vi the event cases where jumbled up, e.g. Snap Image and Save Image where swapped and thus did not work anymore
  • Camera.IMAQdx was not part of the project and the IMAQdx.lib lib is not listed as dependency in .vipc. This might still not work since it still crashed on my machine (I found it confusing it wasn't included but I saw you previously excluded this on purpose so feel free to exclude it again. ;-) )

jhgee avatar Jun 29 '20 08:06 jhgee

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 29 '20 08:06 CLAassistant

Hi John (@jhgee), Thanks for the report. I believe IMAQdx was excluded because this is a paid add-on that not so many people have access to. As for the relinking issues, I'm wondering if a mass compile would have helped in fixing the type propagation in the in-place element structures? Lastly, the event registration being jumbled... I've seen this before and is quite an annoying but in LabVIEW, one that is now supposed to be fixed though, so I'm surprised that you've encountered it in LV2019.

I'll try to reproduce the problems you've encountered and if I do, I'll merge your code to a dedicated LV2019 branch for those who would like to start with this version. The main branch will remain in LV2013...

Thanks for your contribution!

francois-normandin avatar Jul 02 '20 10:07 francois-normandin

Hi Francois,

I just checkout your SMO HAL and unfortunately still in Labview 2020 the event cases are still jumbled. I fixed the manually but it would great to merge this pull request.

Flydroid avatar Sep 27 '21 14:09 Flydroid

Hi @Flydroid ,

Thanks for the report. I think there is no point in keeping the example code in 2013 for backwards compatibility... It's example code after all!

I'll upgrade the base code to a newer version of LabVIEW with those things fixed. I'll probably up it to LabVIEW 2020 so that community edition users have it fully functional without recompilation, which hopefully will limit the risk of event name mangling for that category of users. Not everyone can fix those errors before giving up. I appreciate you took the time to report it.

About IMAQdx, since it's a paid add-on from NI, it won't be included. I'll make sure to create a separate lvproj file so it does not load this dependency.

francois-normandin avatar Sep 27 '21 15:09 francois-normandin

Hi @francois-normandin,

Here am I again, looking at JKI SMO to see if it is the right for a new project. As your HAL example seems to be the only more complicated JKI SMO project available online I would like to ask if you you could have another look at saving your project in Labview 2020 (or similar)?

Thank you!

Flydroid avatar Nov 10 '23 12:11 Flydroid

@Flydroid , sure I'll give this a quick look, although I might but have access to IMAQ license anymore, but let me see if I can find some offline help and get something going.

francois-normandin avatar Nov 10 '23 14:11 francois-normandin

@Flydroid I modified the LV2013 code to prevent the Processes to break when upsaved to LV2020.

There are known issues with User Event mapping that, at the time, I believed had been fixed around LV2012. However, upsaving the SMO.Dictionary classes, Microscope, XYStage and Camera all broke the ordering of the events.

The fix was to wire individual User Events instead of wiring the Cluster of User Events.

As an example... This arrangement maintains the correct mapping when up-saved to LV2020: image

This arrangement did not: image

The code has been modified and saved in LabVIEW 2013 version, but I have verified that it opens and recompiles to LV2020 without breaking.

francois-normandin avatar Nov 11 '23 03:11 francois-normandin

@francois-normandin thank you for the fixes! It's working now!

Flydroid avatar Nov 13 '23 11:11 Flydroid