Vincent Esche
Vincent Esche
@ecstatic-morse I just rebased the Pr onto `master`, which should make the diff clearer now.
> If anything, we'd want to switch to [shorter names](https://rust-lang.zulipchat.com/#narrow/stream/186049-t-compiler.2Fwg-polonius/topic/Reviewing.20Datafrog.20PRs/near/248422230): going from `Val` or `V` to `Value` doesn't add any additional information, but it does make parameter lists longer. Same...
Just checked with v8.2.1. Same crash. Am I doing something wrong?
@ollieatkinson that fixed it, thanks! Mogenerator should probably handle that automatically.
@rentzsch Is there any update or expected timeframe for this getting released?
@s-arash the snapshot tests should now pass and help in catching regressions (e.g. with respect to [build-determinism](https://github.com/s-arash/ascent/pull/34)) in the future. --- It isn't exactly clear to me what your workflow...
> I think checking for the exact expansion output is not the right way to test Ascent, since the exact output can change quite frequently. What should be tested is...
I've updated the tests and gave the `tc` tests some minimal unit tests to illustrate how one would combine expansion with minimal logic tests
In March 2017 [Clue](https://helloclue.com/) and WATTx held a hackathon on Berlin for which, while the proprietary data unfortunately remains unavailable, some of the participants' code is available on GitHub: -...