You generally need to tell us what “not working” means - what is it supposed to do, and what does it do instead?
I can tell you right now though, that Time.time will never be greater than vis, because you’ve just set vis to Time.time. Same thing with nextvis a few lines later. So that’s probably a problem.
I’m trying to make the ToggleAllTargets go from through to false and back every 2 seconds. it doens’t do anything but a weird thing is if I change the value to 6 and then back to 2 while playing it works perfectly
Update() gets called every frame. So right now, every single frame, you are changing the value of “vis” to be the current time plus the value of “invis”. Unless “invis” is negative, that means that “vis” will never be less than the current time.