-
Notifications
You must be signed in to change notification settings - Fork 130
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
Adding more templates and functions #5
Adding more templates and functions #5
Conversation
I have added more templates and functions, noise, primitives and domain operators.
Thanks for the PR! I saw your codes and thought there were a lot of useful functions, and It would make this plugin more powerful. However, I want to keep all codes and examples simple and clean so that everyone who want to use this plugin can understand what this plugin can do in a short time. The PR includes some unnecessary functions (e.g. So, could you separate the PR into small ones that includes only one purpose (ex. adding primitives, adding noise functions, adding operators...) to be able to discuss each function well? |
Hi,
Would you like me to remove the pull request and do individual ones? I can
tidy up and remove a few parts, I included vmax in primitives as a cheap
fix, I think I got an error message about getting to maximum depth of cginc
when using too many references to other cginc files.
I'll take out the code with the license, it's the 2d tiles, it's under
release as general public, I thought it would be useful to tile the out
side of icososphere's.
On the naming convention, I see you are capitalising primitives and
functions, shall everything be capitalised with no reference to it's type,
ie maybe domain operations end with Op to denote an operation, anything
else on the naming convention other than every function starts with a
capital? Does is matter to you about using p or pos for the input of the
distance function?
…On 7 February 2017 at 11:53, hecomi ***@***.***> wrote:
Thanks for the PR!
I saw your codes and thought there were a lot of useful functions, and It
would make this plugin more powerful. However, I want to keep all codes and
examples simple and clean so that everyone who want to use this plugin can
understand what this plugin can do in a short time. The PR includes some
unnecessary functions (e.g. vmax is same as max in HLSL), coding styles
are different from the original ones (e.g. indentation, naming convension),
and some codes seem to have licenses.
So, could you separate the PR into small ones that includes only one
purpose (ex. *adding primitives*, *adding noise functions*, *adding
operators*...) to be able to discuss each function well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIgZJ-03Fcz1q1Z4IRK8Q4MKtcl-1nxcks5raFs4gaJpZM4L4xhX>
.
|
Actually, at this point, I think it might be a good idea for you to improve this plugin in your branch. Because I have a plan to separate some functions into different plugins, for example, the code editor with syntax highlighting, shader template system, and uRaymarching using them as subsystems. Regarding uRaymarching itself, I think it's a little big now, so I'd like to move primitives, fractales, and simple operators into the example layer to keep the library layer simple and extensible so that everyone can write raymarching codes in their own way. |
Ok that's a good idea, are you separating the repository up into smaller
individual components? I'll carry on improving for myself, I still think
colouring needs to be addressed I've got audio controlling the distance
function, so I'm happy! https://www.youtube.com/watch?v=_1e6gZ9q6C4
…On 8 February 2017 at 13:51, hecomi ***@***.***> wrote:
Actually, at this point, I think it might be a good idea for you to
improve this plugin in your branch. Because I have a plan to separate some
functions into different plugins, for example, the code editor with syntax
highlighting, shader template system, and uRaymarching using them as
subsystems. Regarding uRaymarching itself, I think it's a little big now,
so I'd like to move primitives, fractales, and simple operators into the
example layer to keep the library layer simple and extensible so that
everyone can write raymarching codes in their own way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIgZJxK3zt8cM4XroqaAxGwEJAi4FWrDks5rachmgaJpZM4L4xhX>
.
|
Hi Hecomi, your plugin is awesome ! I guess we will all need, at one point, to customize the standard functions, and it is probably a bad idea to directly modify the provided files. So maybe, just like the "Properties" or "Post Effect" tab, a new tab could propose to add some additionnable custom cginc files ? (that could be shared among different materials) What do you think ? |
@pechaut @richhop |
I have added more templates and functions, noise, primitives and domain
operators. And added FFT audio analysis and created method for turning audio into a texture file.