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

Configurable Thread Timeout #32

Open
ryanpoulos opened this issue Oct 8, 2020 · 0 comments
Open

Configurable Thread Timeout #32

ryanpoulos opened this issue Oct 8, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@ryanpoulos
Copy link
Contributor

Is your feature request related to a problem? Please describe.
I'm currently working with Tech 180 on a deployment issue at UTC/Collins. We just found that their sysdef in VeriStand takes so long to initialize and start running that the default thread timeout of 30 seconds is too short to prevent their DSF threads from disconnecting before the system can start running (they have many, many custom devices that do a lot of network comms on init, so it all takes a while). In the short term, I am upping the thread timeout to 180 seconds for them, but this should probably be a configurable thing at some point.

Describe the solution you'd like
Add a ms timeout field to the Thread settings so a user can choose the timeout for idle threads. Or choose no timeout (-1 ms).

Describe alternatives you've considered
Added increased static timeout of 180s, but really that's just a short-term fix.

Additional context
The timeout is currently hardcoded in the Plugin Thread.lvclass:Main.vi (shown below). Code would need to be added to allow this to be a parameter of the Plugin Thread class that can be set during initialize of the DSF.

image

@ryanpoulos ryanpoulos added the enhancement New feature or request label Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant