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

LuaForWindows_v5.1.4-46 #131

Open
mahdi-hajian opened this issue Dec 11, 2020 · 3 comments
Open

LuaForWindows_v5.1.4-46 #131

mahdi-hajian opened this issue Dec 11, 2020 · 3 comments

Comments

@mahdi-hajian
Copy link

math.random() is wrong

for example
math.random(200)
is only 1
or
math.random(200,300)
is only 200

@mahdi-hajian
Copy link
Author

in last version is wrong too
Lua for Windows v5.1.5-52 Released

@TFWol
Copy link

TFWol commented Mar 6, 2023

That's a lua problem.

-- you only get 200
count = 0
while count < 100 do
  number = math.random(200, 300)
  print(number)
  if number == 200 then
    break
  end
  count = count + 1
end
-- works properly
math.randomseed(os.time())

count = 0
while count < 100 do
  number = math.random(200, 300)
  print(number)
  if number == 200 then
    break
  end
  count = count + 1
end

REF:
https://stackoverflow.com/questions/18199844/lua-math-random-not-working

@JJSax
Copy link

JJSax commented Dec 31, 2024

This issue is more than a randomseed issue.

print(os.time()) -- this prints a different time each time. 
math.randomseed(os.time())
for i = 1, 5 do 
  print(math.random(1, 100)) 
end

outputs this, notice the first number on every test. Each test was done several seconds apart from each other to ensure os.time() returned a new number.

-- test 1
42
35
28
93
33
-- test 2
42
68
82
66
65
-- test 3
42
34
91
13
28
-- test 4
42
66
46
87
59
-- test 5
42
99
100
60
91
-- test 6
42
65
9
7
54
-- test 7
42
98
64
80
85
-- test 8
42
30
18
54
17
-- test 9
42
62
36
48
43

Running the same code on several online compilers this issue doesn't happen. Those compilers use Lua 5.4, one allowed me to use down to lua 5.3,2 and the issue was not a thing with that one either.

It's possible this issue existed in Lua 5.1 which this project uses.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants