Skip to content
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

added BallShell class and some noise features #1

Merged
merged 3 commits into from
Aug 14, 2020

Conversation

tomo-makes
Copy link

おうち花火大会、楽しい企画ありがとうございます。我が家でも開催したく参加します!

機能の変更

  • 打上げ時の色を黄色に変更
  • ParticleのHueにノイズを加える(色を一定のレンジでぶらす)
  • ガチャでカラフルな花火が出現(!)

コードの変更

  • Particleクラスを継承し、BallShellクラスを作成
  • 一部の変数名や、実装箇所を変更

自分が書きやすいようコードの構成を変更した部分があり、一旦別ファイル p5_fireworks_variations.pde にしました。

@karaage0703
Copy link
Owner

@tomo-makes さん
PRありがとうございます!グッとリアルになりました。
コード勉強になります。こちらの構成が良いと思うのでp5_firework.pdeに上書きしますね。

masterからpushされているようなので、一旦このPRをマージしてからこちらで別PRで修正しますね。
もし特別な理由がなければ、PRする場合はブランチを切ってからPRの方が何かと良いかもしれません。

@karaage0703 karaage0703 merged commit 7b44526 into karaage0703:master Aug 14, 2020
@tomo-makes
Copy link
Author

@karaage0703 さん
わーいマージ、ご指摘ありがとうございます。masterへのpush失礼しました。。次回(もしできれば!)は気をつけますね。

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