search

BastiaanOlij / gdprocmesh

A procedural mesh generator for Godot build as a GDNative plugin
MIT License
120 stars 13 forks source link

Added split vector #16

Closed leopoldek closed 6 years ago

leopoldek commented 6 years ago

Juggling 3 git branches I almost deleted all my work xD

vector_split

There are also some other nodes I'd like to propose:

Let me know what you think. :)

BastiaanOlij commented 6 years ago

Owh i wanted to make that too, and a surface split to break a surface in its components.

There is a TODO.md in the root in which i’m keeping a list of things, feel free to raise PRs that amend it with your suggestions:)

Cheers,

Bas

On Fri, 24 Aug 2018 at 1:17 pm, Daniel Eliasinski notifications@github.com wrote:

  • Added split vector. Allows user to split an array of vectors into 3 arrays of reals (x, y and z)
  • Renamed Vector to Vector Combine to avoid confusion.

[image: vector_split] https://user-images.githubusercontent.com/15792219/44562584-3c7aa080-a728-11e8-9e8d-17dedca4e4ca.png

There are also some other nodes I'd like to propose:

  • Append: Appends two arrays. Not sure if this is useful but could be nice just in case.
  • Expression Input: allows user to define expressions as inputs
  • Generate: Generate an array of reals using an expression
  • Filter: Evaluates an expression to a boolean. If the bool == true than the real/vector is passed. Else it is blocked.
  • Real Array Input: Allows the user to input an array of reals.
  • Subdivision: Preferably using Catmull–Clark, could be tricky.
  • Add an option to extrude shape to round/smooth the ends of the extrude. Right now they are flat.

Let me know what you think. :)

You can view, comment on, or merge this pull request online at:

https://github.com/BastiaanOlij/gdprocmesh/pull/16 Commit Summary

  • Added split vector

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/BastiaanOlij/gdprocmesh/pull/16, or mute the thread https://github.com/notifications/unsubscribe-auth/AB2vabEA18CiroNHH7383oRsOv4JKyhNks5uT3A4gaJpZM4WKrlq .

-- Kindest regards,

Bastiaan Olij

https://www.facebook.com/bastiaan.olij https://twitter.com/mux213 https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw https://github.com/BastiaanOlij

BastiaanOlij commented 6 years ago

Ah, this conflicts with your other PR. Can you rebase it? Then I can merge it:)

leopoldek commented 6 years ago

Uhh don't merge this yet I did it on Github but I don't like the way it did it I'll do it locally soon

leopoldek commented 6 years ago

Ok that's better its ready.