Make 'translate' an alias for 'put-image'

bootstrapworld / curriculum

6 stars 7 forks source link

Make 'translate' an alias for 'put-image' #1454

Open schanzer opened 1 year ago

schanzer commented 1 year ago

[x] Find-and-Replace put-image with translate in all our materials
[x] make translate available in WeScheme
[x] Make translate as an alias for put-image in contexts
[ ] Make sure framing of the function is updated appropriately
[ ] Update starter files
[ ] Update desmos
[ ] Decide whether put-image or translate will be used in Chinese Flag starter file, remove the other, correctly save files in google drive and update the link in starter-files.js
[ ] add translate to .langtable.js
[ ] use add-to-lang to add translate to contracts-tables-displays

ds26gte commented 1 year ago

do you mean all occurrences of translate in code fragments should become put-image?
do you mean this should happen only for Pyret, when converting from our common-s-exp language input?

schanzer commented 1 year ago

I'm thinking this would happen in our Bootstrap-A and Bootstrap-DS contexts, so as to be entirely transparent (I would have to update WeScheme as well).

And yeah, we'd do a global F-&-R in our materials to remove put-image (perhaps preserving it for a year or two in the contracts list)

schanzer commented 10 months ago

Luke is still not responding to email. Moving this to Feb 1

schanzer commented 9 months ago

The find-and-replace for this issue has been completed, and resides in the put-image2translate branch. Adding this to our curriculum team meeting agenda, to see if we want to move forward.

flannery-denny commented 9 months ago

@schanzer Take a look at this Chinese Flag starter file! translate and put-image don't put the images in the same place!

schanzer commented 9 months ago

@flannery-denny good catch!

It looks like Joe may have made translate and alias for place-image instead of put-image. I'll poke him.

flannery-denny commented 2 months ago

@schanzer any update on this?

schanzer commented 2 months ago

@flannery-denny pyret has it now (albeit too late for fall2024). I've been trying to keep the put-image2translate branch in sync withmaster`, so this should be fairly low-hanging fruit

flannery-denny commented 3 weeks ago

@schanzer starting to work on 2025 datasets. Do you have a new context with translate built in yet?

schanzer commented 3 weeks ago

@flannery-denny That to do item has already been checked off. Are you saying that Translate isn't working for you? Can you send me a link to the offending file?

flannery-denny commented 3 weeks ago

@schanzer just wanted to make sure I was using the right context for the future. My notional machine was broken... It works :)