Skip to content

fixed wrong get indicator label and added reverse 1d array to ctrl se…#421

Open
pettaa123 wants to merge 7 commits intoni:masterfrom
pettaa123:correct_get_accessor_label_ident_in_genoneof_get_accessor
Open

fixed wrong get indicator label and added reverse 1d array to ctrl se…#421
pettaa123 wants to merge 7 commits intoni:masterfrom
pettaa123:correct_get_accessor_label_ident_in_genoneof_get_accessor

Conversation

@pettaa123
Copy link
Copy Markdown
Contributor

…arch

GetOneOf_Get_Accessor.vi search for wrong label Get_Indicator, instead of Get_Val as in OneOfTemplate.lvclass:GetTemplate.vi
This worked since Get_Val was always last in ctrl array.
Because of this I added a reverse 1d array to ctrl array, since Get_Val and Set_Val appear to be always last.

image

image

image

…arch

Signed-off-by: Jens-Peter Tuppatsch <tuppatsch@gmx.de>
…onOneOf_Get-Set_Accessor throw error if control-indicator not found

Signed-off-by: Jens-Peter Tuppatsch <tuppatsch@gmx.de>
@pettaa123
Copy link
Copy Markdown
Contributor Author

I didn't know about the tabbing order, I learned that. Thanks

@jasonmreding
Copy link
Copy Markdown
Collaborator

@pettaa123 It appears one or all of the VIs from this PR were saved in LV 2020 rather than LV 2019.
https://github.com/ni/grpc-labview/actions/runs/14582170845/job/40977143664?pr=421#step:6:32

Copy link
Copy Markdown
Collaborator

@jasonmreding jasonmreding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resubmit VIs saved in LV 2019.

Tuppatsch and others added 4 commits April 23, 2025 09:08
…arch

Signed-off-by: Jens-Peter Tuppatsch <tuppatsch@gmx.de>
…onOneOf_Get-Set_Accessor throw error if control-indicator not found

Signed-off-by: Jens-Peter Tuppatsch <tuppatsch@gmx.de>
…or' of https://github.com/pettaa123/grpc-labview into correct_get_accessor_label_ident_in_genoneof_get_accessor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants