Jelle van Assema
Jelle van Assema
Relevant vid: https://www.youtube.com/watch?v=AbSehcT19u0
This is sadly a feature. In `structure` whitespace tokens are thrown out entirely. Once compare50 finds a match in code between two submissions, it will try to "expand" that match...
Do you have the relevant slug by any chance?
This happens because the cs50 library sets the `sys.excepthook` itself unconditionally to a lambda function. That famously are not pickle-able, because where'd be the fun in that. As it happens,...
https://github.com/cs50/check50/tree/unpickleable_attributes
The current comparison methods don't do line-by-line comparison, but use k-grams (continuous sequences of tokens) instead. In this model it can be that short lines don't match each other, because...
The difficult thing with a diff is alignment. The code may be shuffled a bit, and odds are similar pieces of code appear in different parts of submissions. And it's...
Hi, thanks for the PR! One quick note from me, probably best to not rely on assertions here and to just use an if-statement instead. As [assertions can be disabled...
Current thinking is that the views should look the same, no matter the machine / internet connection. But it's probably a good idea to use commonly available (web safe) fonts...
Tricky one there. In short, the current noise threshold is too high for function definitions. These definitions carry relatively few tokens (especially in speller) and are by compare50's current standards...