I think there are two problems here—a narrow one that falls within the scope of this pull request, and a broader one that deserves its own pull request.
Narrow: semantics issue
The…
Now that I've merged pull request #81, the branch for this pull request (curvature-regulators) needs to be rebased onto main.
Or maybe not! I just noticed that the merge control panel says…
Now that I've merged pull request #81, the branch for this pull request (curvature-regulators) needs to be rebased onto main. If I'm the only one with a local copy of this branch, I'd be happy…
I've made the changes we discussed during our meeting:
- Use field init shorthand in the regulator…
We approved my revisions during today's meeting, so I'll merge!
If you're happy with the revisions I made in commit 664b3d8, I'll go ahead and merge.
I haven't figured out how to do it with cargo test, however.
I think the right way to do this would be to use the [-C flag](https://doc.rust-lang.org/cargo/commands/cargo-test.html#common-op…
This is useful information, and it works on my system. I've streamlined the instructions a bit (commit 664b3d8) using the fact that only one step depends on which folder you're in.