[SOLVED] Why Is My GUI Giver is Giving Everyone The GUI When 2 People Should be Getting The GUI?

I was setting the variable value, not the actual value. This is considered solved

local player1 = game.ReplicatedStorage.ServerRoundStats.Player1.Value
local player2 = game.ReplicatedStorage.ServerRoundStats.Player2.Value
local playercount = 0
local csc = game.ReplicatedStorage.Remotes.CStartRoundC
local cs = game.ReplicatedStorage.Remotes.CStartRound
local afked = game.ReplicatedStorage.Remotes.AFKPlayer
game.ReplicatedStorage.Remotes.StartRound.Event:Connect(function()
    for i,v in pairs(game.Players:GetChildren()) do
        playercount = playercount + 1
    end
    repeat
    for i,v in pairs(game.Players:GetChildren()) do
        if player1 == nil then
            local random = math.random(1,playercount)
            if random == 1 then
                player1 = v.Name
                v.Role.Value = "P1"
            end
        else
            if player2 == nil then
                local random2 = math.random(1,playercount)
                if random2 == 1 then
                    player2 = v.Name
                    v.Role.Value = "P2"
                end
            end
        end
    end
until player1 ~= nil and player2 ~= nil
for i,v in pairs(game.Players:GetChildren()) do
    if v.Role.Value == nil then
        cs:FireClient(v)
    else
    csc:FireClient(v)
    end
end
end)

afked.OnServerEvent:Connect(function(plr)
    if player1 == plr.Name then
        player1 = nil
        for i,v in pairs(game.Players:GetChildren()) do
        local random = math.random(1,playercount - 1)
            if random == 1 then
            if v.Name ~= plr.Name then  
            player1 = v.Name
        v.Role.Value = "P1"
        else 
        repeat
        for i,v in pairs(game.Players:GetChildren()) do
        local random = math.random(1,playercount - 1)
            if random == 1 then
            if v.Name ~= plr.Name then  
            player1 = v.Name
        v.Role.Value = "P1"
        end
        end
        end
        until player1 ~= nil and player1 ~= plr.Name
        end
        end
    end
    else
        if player2 == plr.Name then
                    player2 = nil
        for i,v in pairs(game.Players:GetChildren()) do
        local random = math.random(1,playercount - 1)
            if random == 1 then
            if v.Name ~= plr.Name then  
            player2 = v.Name
        v.Role.Value = "P2"
        else repeat
        for i,v in pairs(game.Players:GetChildren()) do
        local random = math.random(1,playercount - 1)
            if random == 1 then
            if v.Name ~= plr.Name then  
            player2 = v.Name
        v.Role.Value = "P2"
        end
        end
        end
        until player2 ~= nil and player2 ~= plr.Name
        end
        end
        end
        else
     return
end     
    end
end)

The LocalScript (Should only give the 2 people from the last script the GUI)

local reg = game.ReplicatedStorage.Remotes.CStartRound
local chose = game.ReplicatedStorage.Remotes.CStartRoundC
local sendreq = game.ReplicatedStorage.Remotes.WaitForCode
local plr = game.Players.LocalPlayer
local remotevote = game.ReplicatedStorage.Remotes.StartVoteTimer
local player1 = game.ReplicatedStorage.ServerRoundStats.Player1.Value
local player2 = game.ReplicatedStorage.ServerRoundStats.Player2.Value

reg.OnClientEvent:Connect(function()
--nothing needs to be done--
end)

chose.OnClientEvent:Connect(function()
    if plr.Name == player1 or player2 then
    local gui = game.ReplicatedStorage.Guis.AudioCode:Clone()
    gui.Parent = plr.PlayerGui
    sendreq:Fire()
end

end)

remotevote.Event:Connect(function()
    local gui = game.ReplicatedStorage.Guis.VotingGUI
    gui.Parent = plr.PlayerGui
end)