-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Review the eraser tool #65
Comments
I made a notebook (on ReMarkable version beta 3.3) that looks at the eraser: eraser testing.zip In it, there are three squares and three lines of text.
In neither cases does (2) the eraser at the back of the pen fail, but what's really strange here is that (3) only the text has visible eraser marks when the tool is used. |
Here's a more carefully constructed file. Oddly enough it has fewer obvious issues. I took a photo with my phone before and after erasing, to ensure I didn't forget copying a drawing before erasing it. What you can see in the remarks output is that everything is erased as expected. However, in the first vertical table bar, there are two holes in the 3rd and 4th column (next to "hello" and the scribbles underneath it). Weird! Before: After: Remarks' output: The notebook: |
I found that the color codes of the erasers are incorrect. Two of them have color-code 0, which is black. The other one has color-code 2, which is white (correct) I'm not sure why. I suspect we can safely override the eraser's color with WHITE in all cases. I have a slight suspicion that ReMarkable has a global So if you have an eraser, then select a tool and put the color to black and go back to the eraser, the eraser could have a color-code of black, even though this value is meaningless for the eraser. This is just a suspicion though. |
This issue occurs because the calculation for the eraser width produces very large values for certain elif tool == "Eraser":
w = 1280 * w * w - 4800 * w + 4510
# cc = 2 I'm not sure where these values come from, but with the input value of >>> 1280 * w * w - 4800 * w + 4510
362.7999999999993 |
Ref #63 (comment) and #63 (comment).
The text was updated successfully, but these errors were encountered: