NevinBR

It looks like `_minMaxHeapIsMinLevel` is always called with a `+1` at the call-site, as in: ``` _minMaxHeapIsMinLevel(index + 1) ``` Would it make sense to move the `+1` into the...

> the calculation is ultimately based on the `count` of items, not the index. That seems…needlessly pedantic to the point of causing confusion to readers. The caller is working with...

I wonder—is `sincos` optimized enough to make it worthwhile to generate *pairs* of values from a normal distribution? And how about, um, `sincos(piTimes:)`? ``` extension Real { static func boxMullerTransform(_...

My question was not whether we need to write `sincos`. My question was whether we should design the API to include a function which generates pairs of normally-distributed values.

Would basic number-theory functions be in scope for this module? * gcd * lcm * Bézout coefficients * Modular inverse

I’d like to start working on this. What’s the recommended process? Should there be a dedicate branch for work on this module?

What’s our file-naming strategy? I see you’ve created “Rotated.swift”. Do we want all files to be that small and single-purpose, or should we rename it something like “BitwiseOperations.swift”? The first...

I agree that we should stick to ASCII operators for the foreseeable future. However, just as a point of reference, I have been using a number of `√` operators in...

> So the set of things that could conform is basically the reals and the complex numbers There could also be an `ElementwiseMatrix` type, which conforms using elementwise operations when...

It’s worth noting that in Swift today, one cannot extend a protocol composition. This suggests that either we should make a refining protocol instead of a typealias, or we should...