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

VoiceAttack, Start Gear Announcement, setting SRA_loop false still loops #136

Closed
samkatakouzinosracing opened this issue Jan 9, 2018 · 4 comments

Comments

@samkatakouzinosracing
Copy link

commented Jan 9, 2018

I didn't want to hear gear changes through SRA.
I set the boolean to false.
Gear changes were still announced.

I changed logic to set and test for integers (0,1).
This works better.

@jfgilliam

This comment has been minimized.

Copy link
Contributor

commented Jan 23, 2018

You can also remove it from the Start Spotter command.

@samkatakouzinosracing

This comment has been minimized.

Copy link
Author

commented Jan 24, 2018

TL:DR; Please change global variable SRA_Loop to command scope ~SRA_Loop for "Start Flags", "Start Gear Announcement", "Start Leader Change", "Start Pit Road".

Ok, page 158 of VA manual talks about "Advanced Variable Control (Scope)".
Variables can be declared globally (as you are doing), profile scoped and command scoped.
Global variables (variables available to all commands and profiles) have no prefix.
Profile scoped variables are prefixed by ">" and are available to all commands within the profile but not other profiles.
Command scoped variables are prefixed by "~" and are available to all command entries and sub-commands. They are not accessable to other commands.
So! I changed all SRA_Loop variable declarations and references to ~SRA_Loop in "Start Flags", "Start Gear Announcement", "Start Leader Change", "Start Pit Road".
Changing to Command scope variables now works as I intended.

@jfgilliam jfgilliam reopened this Jan 25, 2018

@jfgilliam jfgilliam self-assigned this Jan 25, 2018

@jfgilliam jfgilliam added this to the Version 1.6 milestone Jan 25, 2018

@jfgilliam

This comment has been minimized.

Copy link
Contributor

commented Jan 25, 2018

Probably should do this to all my variables, but that's a lot of work. I name most of the uniquely to prevent this problem. I don't think VA had local variables when I first built this profile, but I don't remember.

@jfgilliam jfgilliam changed the title VoiceAttack, Start Gear Annoucement, setting SRA_loop false still loops VoiceAttack, Start Gear Announcement, setting SRA_loop false still loops Jan 25, 2018

@jfgilliam jfgilliam closed this in 97a9934 Feb 2, 2018

@samkatakouzinosracing

This comment has been minimized.

Copy link
Author

commented Feb 3, 2018

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.