This repository has been archived by the owner on Apr 4, 2023. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 4
importing tempfile breaks expected behaviour of std / random #13
Comments
please consider this module is deprecated since |
Ok we will switch to temp files from stdlib. Should I make a Pr to add this deprecation notice to the readme? |
I will do it soon, and remove it from Nimble directory too. Thank you! |
Well if you can wait to remove it from nimble until we make the fix on our side... otherwise it will break anyone trying to install nimib :) |
Ofcourse I can wait :) |
And there might be other libraries depending on it, maybe you could add a deprecation notice to appear when the library is installed/used and remove from nimble after a deprecation period? |
its ok, so I will archive this project only. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
module random from stdlib requires a call to randomize in order to have random results that change at every call. Indeed the following:
produces always the same number when run again (on my machine: 3).
If tempfile is imported instead the behaviour changes, since indeed tempfile calls
randomize
in its global space. so the following:will produce always different numbers.
Is it necessary to have this randomize call or could it be removed (or hidden behind the switch and documented this behaviour)?
Btw, I have been happily using tempfile as a dependency in nimib since its beginning, thanks for creating this!
The text was updated successfully, but these errors were encountered: