Created menus as source code, not as VIPM artifacts. Updated build s…#153
Created menus as source code, not as VIPM artifacts. Updated build s…#153niACS wants to merge 1 commit intoni:developfrom
Conversation
|
If the intention of this change was that the new mnu files are only for AF 2024 for 2020 package to have these mnu files, then I would rather have subfolders under Menus where both these new mnu's and the existing mnu's go to. |
I edited the palettes using the native LabVIEW palette editing tools, and removed that subscript. It does not appear when I run the LabVIEW IDE. How is it still there, and what tool do I need to use to make that change? |
How do I edit that from the LabVIEW IDE? |
These mnu files are intended to replace and extend the mnu files that ship with 2026 Q1. These are the correct files for that release of Actor Framework. |
|
It looks like I hit some "gotchas" in palette editing regarding things that are not accessible through the normal "Advanced » Edit Palette Set..." menu option. Are there other editing tools with which I am not familiar, and can they be made available? |
The Menus directory represents LabVIEW's Menus directory. We should not create hierarchy in source that doesn't match what would ship. If you feel like you need more qualification on the purpose of that directory, you could make an additional level above Menus but you shouldn't mess with Menus and below. |
You can use the "Write Palette.vi" present in LabVIEW to edit the palette file. That gives you fine grained control of editing the palette file. You can use the mnu files that are currently directly under the Core folder as the source for the finer details of the new palette files. Also after the change we should probably just have version |
|
Since @niACS is OOO for the next couple weeks, @dhooks1 and I will look into it on the SteerCo side of things. |
|
The .mnu files referenced in this PR still appear to have a "2024 for 2020" submenu (that's missing). I'm creating a new PR to address the issues listed by @shivaprasad-basavaraj. Link to follow. |
|
I can recreate those changes in the other PR since I don't have access to the branch in yours @niACS . |










…pecification.
GitHub Issue for the Pull Request
Link to the relevant GitHub issue
GitHub Discussions Related to this Pull Request
None
Checklists
Completing these checklists ensures that your PR meets our build server requirements on the first attempt. If you're unable to complete any of the following checks, please submit your PR as a draft to the best of your ability. We will provide any clarification you may need after posting it.
I built and installed the VI package locally. I don't know the state or process for using the Powershell tool.
Summary of Changes
I added the appropriate menus to the correct menu folders, and disabled menu creation in the VIPM build specification. The following were modified or updated:
actorframework.mnu
_actorframework_experttools.mnu
_actorframework_msgqueuepair.mnu
_actor_framework_testsupport.mnu (new)
actorframework_controls.mnu (new)
Reason for Change
2026 Q1 shipped with the old palettes. These are the correct ones.
Visual Aids
Include any visual aids that would help the reviewer gain additional context.
Additional Information
Include any further details that may assist the reviewer in understanding the context of this PR.
Testing
This section describes the automated and manual tests performed for this bugfix/feature.
Manual Tests
Manual inspection. Yep, the new palettes are there.