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

Smooth Quad Distortion #4288

Closed
1 of 11 tasks
Vamoss opened this issue Feb 16, 2020 · 2 comments · Fixed by #4901
Closed
1 of 11 tasks

Smooth Quad Distortion #4288

Vamoss opened this issue Feb 16, 2020 · 2 comments · Fixed by #4901

Comments

@Vamoss
Copy link

Vamoss commented Feb 16, 2020

Most appropriate sub-area of p5.js?

  • Color
  • Core/Environment/Rendering
  • Data
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Other (specify if possible)

New feature details:

Quad distortion now:
https://editor.p5js.org/Vamoss/sketches/vWdWD1q4
image

Smooth squad demo suggestion:
https://editor.p5js.org/Vamoss/sketches/1iaogZo2
image

@stalgiag
Copy link
Contributor

I like your example. I don't think we will be adding more features before the 1.0 release but this is a simple enough addition to consider adding in some capacity in the future.

Do you think it would be possible to rewrite this in such a way that the existing quad function could take detailX and detailY optional parameters?

This feature sits in an odd place because it is a combination of plane and quad in that it offers the independent corner pin of the quad arguments with the flexible detail of the plane arguments.

@Vamoss
Copy link
Author

Vamoss commented Mar 19, 2020

Great, I saw 1.0 was released, congratulations!

Totally agree, the quad function could be already replaced by this one, as you can see, the default detailXand detailY is 8, if you change the default to 2, the result will be the same of quad function.

Vamoss pushed a commit to Vamoss/p5.js that referenced this issue Nov 16, 2020
@Vamoss Vamoss mentioned this issue Nov 16, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants