Skip to content
This repository has been archived by the owner on Jul 5, 2022. It is now read-only.

Port of CC123_2 to Processing #866

Merged
merged 13 commits into from Dec 2, 2018
Merged

Port of CC123_2 to Processing #866

merged 13 commits into from Dec 2, 2018

Conversation

gruselhaus
Copy link
Member

Choo Choo 🚂

// Part 2: https://youtu.be/A0NHGTggoOQ
// https://thecodingtrain.com/CodingChallenges/123.2-chaos-game

ArrayList<PVector> points = new ArrayList<PVector>();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't feel strongly either way but would it make sense to just have a plain array here that is created in setup() since the number of points is fixed? I guess this is less flexible for a version where the points might be interactive?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes totally sense. Thanks for this advice. I'm still learning 😀

Copy link
Member

@shiffman shiffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually I meant like this! This new suggestions feature is fun! ❤️

shiffman and others added 7 commits November 29, 2018 21:41
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
…e_2/CC_123_ChaosGame_2.pde

Co-Authored-By: gruselhaus <nico@gruselhaus.com>
@shiffman shiffman merged commit 44c783b into CodingTrain:master Dec 2, 2018
@shiffman
Copy link
Member

shiffman commented Dec 2, 2018

Yay, thank you!

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

Successfully merging this pull request may close these issues.

None yet

3 participants