-
Notifications
You must be signed in to change notification settings - Fork 3
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
canvas size is 101x101 pixels #3
Comments
Thanks so much for pointing this out! I wasn't aware; gonna fix this. (And I assume if |
The original dbn covered in the book had no "size" command - all sketches were 101x101 pixels. 10201 addressable pixels is definitely idiosyncratic by today's standards, but maeda was very explicit that he wanted the dbn canvas to have a logical center. Early sketches were encouraged to be designed around this idea of an x=50 centerline, and in this port they appear slightly left of center. I realize this might seem petty, but if people really are interested in the history of these toolkits, then I feel it is worthwhile to try to prevent any accidental "historical flattening" as its precisely these rough edges which make the language historically interesting and without them the language's quirky character and some of the constraints/decisions that shaped it are also lost. |
I made a PR to implement this change, took 10 seconds :) https://github.com/LingDong-/dbn.js/pull/5
|
I think I have fixed this, so sorry for the delay, @dribnet In addition to @gingerbeardman 's change, I commented out these 1px shifts: https://github.com/LingDong-/dbn.js/blob/main/dbn.js#L294 I think when I first coded this, I noticed that the point (100,100) is visible in the original version, so I thought that the canvas starts at (1,1). Turns out it is instead because the canvas is 101x101. This test now looks correct: https://github.com/LingDong-/dbn.js/pull/5#issuecomment-2495668212 The change is live: https://dbn.glitch.me/ It's been a while and my DBN is a bit rusty, please let me know if if I missed anything! Thanks, and sorry again for the delay. |
In dbn
x=0
andy=0
are valid on-screen viewable locations and the addressable canvas size is 101x101 pixels. The interpreter is currently using a canvas size of 100x100 and cropping these out, removing a single line of pixels from both the bottom and left of the image. This is effecting both the interactive version and the exported gif files.The text was updated successfully, but these errors were encountered: