Yes, this is one of the recommended ways to organize tests in Rust!
Are the _1 and _2 suffixes the number of arguments?
Yes—log_0 through [log_7](https://docs.rs/web-sys/latest/web_sys/conso…
I think it's too early to design a formal logging system. I've been adding and removing log messages pretty freely as I test new features and track down bugs. I just started a [wiki page](wiki/Logg…
The PR update will add a comment that explains the index field's meaning, as described in this comment, and reminds us to…
When an assembly with n elements is sent to the engine, each element gets an index from 1 to n, which tells you which column of the configuration matrix it goes in. The index is assigned and…
I assume the two
usizes are some sort of references to elements. If a primitive type is being used with semantic baggage, typedef it or something like that so that you can write e.g.,…
This seems to assume that there will be two elements constrained.
Right now, there's only one kind of element (a sphere) and one kind of constraint (fixing the generalized angle between two…
Name choice? These are the two elements constrained by the constraint? We should prefer an entire word or short phrase that makes it clear. constrains would work for me, as would appliesTo.…
Same comments go for rep here as in Constraint. Maybe it's "representative"?
It's for "representation"—a vector representation of the element, acted on by a linear representation of the 3d…
If you go into
engine-proto/gram-test, launch Julia, and theninclude("irisawa-hexlet.jl"), you should see that it prints basically the same loss history until the last few steps, when the…
OK, went through the code. Since this prototype seems like it has a chance to become the start of the actual code, I don't think it's too early to start trying to use some hygienic naming and…
Are they theoretically identical, with differences only occurring for numerical reasons?
Yes. In both cases, we're trying to solve a problem like
$$\left[\begin{matrix} a & b \ c & d…
Please fix the warning
Done—by compiling engine::point only in test mode, where it's already in use. I'll update the PR once I do the other requested changes.
if there is a compiler…
