你的回答有3个问题:q中操作的优先级、[-1;1]中如何生成随机变量、变量sq上的条件。
操作的优先级:
在下面提醒一下,a1 和 a2 是不同的:
u:rand[1f];
a1:-1+2.0*u;
a2:2.0*u-1;
u:0.5
a1:0f
a2:-1f
u:0
a1:-1f
a2:-2f
在[-1;1]中生成一个随机变量:
使用以下内容:
a:-1+2.0*rand[1.0];
b:-1+2.0*rand[1.0];
sq 上的条件:
您确实考虑了sq >= 1 条件,但如果sq = 0 也有问题,因为您必须在最后一步除以sq。
此外,在您的实现中,您计算了两次 a,其中 1) 不是最优的,2) 与方法不一致,因为在最后一次计算 sq 时必须使用相同的 a步骤,后者屈服于非常大的数字。我从wikipedia page 中获得了一些灵感,他们建议在不满足上述条件的情况下重新生成sq。因此在下面的实现中函数中的递归调用:
.stat.GetOneGaussianByBoxMuller:{
a:-1+2.0*rand[1.0];
b:-1+2.0*rand[1.0];
sq:(a*a)+(b*b);
if[(sq>=1)|(sq=0);
:.stat.GetOneGaussianByBoxMuller[];
];
:a*sqrt[(neg[2]*log[sq])%sq];
};
您现在可以查看通过绘制以下数据集的直方图生成的数据:
([]val:{:.stat.GetOneGaussianByBoxMuller[]} each til 100000)
编辑:
您实际上可以通过为每个函数调用生成 2 个随机数来实现更高效的实现,如下所示:
.stat.GetOneGaussianByBoxMuller:{
a:-1+2.0*rand[1.0];
b:-1+2.0*rand[1.0];
sq:(a*a)+(b*b);
if[(sq>=1)|(sq=0);
:.stat.GetOneGaussianByBoxMuller[];
];
:(a*sqrt[(neg[2]*log[sq])%sq];b*sqrt[(neg[2]*log[sq])%sq]);
};
([]val:raze {.stat.GetOneGaussianByBoxMuller[]} each til 50000)
这个实现需要 150 毫秒来生成 100000 个正态分布的随机数,而上面的实现需要 245 毫秒。