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

Add link to the Julia binding #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

findmyway
Copy link

This package is awesome!

The APIs are really concise, which makes it very easy to provide a Julia binding with the C API.

@Auburn
Copy link
Owner

Auburn commented Aug 13, 2022

Hi, I had a quick look at your Julia API, it doesn't look like it supports setting node variables or creating node trees from "Encoded Node Tree" strings. Without support for these it means most of the features in the FastNoise2 library are inaccessible.

I don't want to include a link to this unless there is full API support. Have a look at the C# bindings, you should be able to copy how it is implemented there.

Thanks

@findmyway
Copy link
Author

OK, actually the bindings are all available at https://github.com/JuliaReinforcementLearning/FastNoise2.jl/blob/master/src/LibFastNoise.jl . This means it provides the full API support in a nutshell. It's just that I added some extra handy functions for generating noise to make the usage more Julian.

I'll take a look into the C# code to see if I can improve further.
Thanks for your feedback!

@Auburn
Copy link
Owner

Auburn commented Aug 13, 2022

Just having access to the raw C API isn't very usable, since the FastNoise2 API provides the node metadata as IDs.

See the C# example usage here: https://github.com/Auburn/FastNoise2Bindings/blob/master/CSharp/test/BitmapGenerator.cs

It's possible to set a node variable using the variable name as a string

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