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

Fix random numbers unavailable warning #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lkoehl
Copy link

@lkoehl lkoehl commented Mar 14, 2023

When using the package with XeLaTeX, the package warns you about the random selection of exercises. The last sentence is missing a 'not'. I also improved the sentence for better readability.

@muzimuzhi
Copy link

XeTeX has learnt random primitives since texlive 2019, see texlive release history, 2019

9.1.16 2019
XeTeX: New primitives ..., \normaldeviate, \uniformdeviate, \randomseed; ... .

and the XeTeX reference guide (Feb 19, 2023), sec. 8.3 Randomness .

One can double check by compiling below using xelatex

% !TeX TS-program = xelatex
\documentclass{article}
\usepackage{pgffor}

\begin{document}
\foreach \i in {1, ..., 10}
  { \fpeval{randint(10)}, }
\end{document}

Therefore the relevant warnings can be safely removed from xsim.

@muzimuzhi
Copy link

Therefore the relevant warnings can be safely removed from xsim.

Or change it to "Please be aware that random selection of exercises is not available in XeLaTeX older than 2020 (exclusive)."

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

Successfully merging this pull request may close these issues.

2 participants