Skip to content

Update/Updated Generative Design Examples #774

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

Merged
merged 28 commits into from
Dec 7, 2018

Conversation

joeyklee
Copy link
Contributor

@joeyklee joeyklee commented Dec 3, 2018

Hello!

The Generative Design group recently did a review and update of the Generative Design Examples (see: generative-design/Code-Package-p5.js@c40b36e). We'd love to do a bulk update of the examples in the p5 web editor when you get a chance!

If you don't mind running the npm run fetch-examples-gg so that all the sketches in the generative-design account are updated, that would be super. We currently have our examples running directly on our site and include a link to the P5 editor for people to jump into the code if they are curious :)

Thanks so much @catarak 🌈❤️✨!

@catarak
Copy link
Member

catarak commented Dec 6, 2018

I noticed that in the PR you linked, it uses p5 version 0.7.2. Do you want to change the p5 version here to 0.7.1 or 0.7.2?

@joeyklee
Copy link
Contributor Author

joeyklee commented Dec 7, 2018

@catarak - ah yes! Thanks so much for catching this one. Yes, the latest version would be best. I've commit the changes in e261b50 Thanks so much!

@catarak
Copy link
Member

catarak commented Dec 7, 2018

great! i'll merge this in now.

@catarak catarak merged commit 5bcb511 into processing:master Dec 7, 2018
@joeyklee
Copy link
Contributor Author

joeyklee commented Dec 7, 2018

Super! Thanks so much @catarak 🌈🙌

@catarak
Copy link
Member

catarak commented Dec 7, 2018

@joeyklee just ran the script to update the examples, let me know that it worked!

@joeyklee
Copy link
Contributor Author

joeyklee commented Dec 7, 2018

@catarak - thanks! I switched from raw git to githack I think there might be some issues with the way that images are served up via those URLs.

E.g. https://editor.p5js.org/generative-design/sketches/P_4_1_2_02 doesn't load the image via:

Just flagging this in case you need to switch any of your other examples over from rawgit to something else. I will look into the fix here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants