Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Root Actor Class is Loaded.vi to AFPP Shared.lvlib #50

Open
niACS opened this issue Feb 17, 2025 · 1 comment
Open

Move Root Actor Class is Loaded.vi to AFPP Shared.lvlib #50

niACS opened this issue Feb 17, 2025 · 1 comment

Comments

@niACS
Copy link
Collaborator

niACS commented Feb 17, 2025

Describe the bug
Root Actor Class is Loaded.vi is a member of Message Maker.lvlib. It is invoked by another member of Message Maker.lvlib and by Add Actor Interface.lvlib:CreateNewWizard_IncludeItem.vi.

Root Actor Class is Loaded.vi is the ONLY member of Message Maker.lvlib invoked by a member of Add Actor Interface.lvlib. I want to move it to AFPP Shared.lvlib, to simplify library dependencies for Add Actor Interface.lvlib.

Message Maker.lvlib already depends on AFPP Shared.lvlib, so we would not increase dependency complexity for that library.

To Reproduce
Steps to reproduce the behavior:

  1. Open a blank project.
  2. Add Add Actor Interface.lvlib to the project
  3. Note that Message Maker.lvlib appears under Dependencies.

Expected behavior

  1. Open a blank project.
  2. Add Add Actor Interface.lvlib to the project
  3. Message Maker.lvlib should NOT appear under Dependencies.

Screenshots
none

Desktop (please complete the following information):
All

Smartphone (please complete the following information):
n/a

Additional context
We should strive to eliminate unneeded complexity in the provider tools wherever possible, to facilitate further development.

@niACS
Copy link
Collaborator Author

niACS commented Feb 17, 2025

The document "Actor Framework Project Providers Design Reference" should be updated after this bug is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

2 participants