-
-
Notifications
You must be signed in to change notification settings - Fork 128
v1.0 Release #122
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
base: master
Are you sure you want to change the base?
v1.0 Release #122
Conversation
Merge latest master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This release modernizes the SIMD dispatch pattern, adds scalable feature and output range controls, updates configuration paths, and renames the “NoiseTool” to “Node Editor”.
- Replace old
FS_T<>
templates withFastSIMD::DispatchClass<>
specializations and force-inline annotations - Introduce
ScalableGenerator
andVariableRange
for feature scaling and output range handling - Migrate config and export headers to
Utility/Config.h
&Utility/Export.h
, remove obsolete FastNoise_Config/Export - Update CMake presets, workflow files, and README to refer to the new “Node Editor” tool
Reviewed Changes
Copilot reviewed 105 out of 105 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
include/FastNoise/Generators/DomainWarpFractal.inl | SIMD dispatch overhaul for domain-warp fractals |
include/FastNoise/Generators/DomainWarpFractal.h | Metadata updates and new FeatureSet dispatch signature |
include/FastNoise/FastNoise_C.h | C API signatures updated (frequency parameter removed) |
CMakePresets.json | Rename and normalize “tools” presets |
README.md | Rename “NoiseTool” references to “Node Editor” |
Comments suppressed due to low confidence (4)
include/FastNoise/Generators/DomainWarpFractal.h:37
- Typo in class name 'DomainWarpFractalIndependant' – consider renaming to 'DomainWarpFractalIndependent' for correct spelling.
class FastSIMD::DispatchClass<FastNoise::DomainWarpFractalIndependant, SIMD> final : public virtual FastNoise::DomainWarpFractalIndependant, public FastSIMD::DispatchClass<FastNoise::Fractal<FastNoise::DomainWarp>, SIMD>
CMakePresets.json:55
- [nitpick] Inconsistent capitalization: change displayName to "Tools Debug" to match style.
"displayName": "tools Debug",
CMakePresets.json:60
- [nitpick] Inconsistent capitalization: change displayName to "Tools Release" for consistency.
"displayName": "tools Release",
include/FastNoise/FastNoise_C.h:10
- Removed the 'frequency' parameter from
fnGenUniformGrid2D
(and related) – this is an API-breaking change. Consider preserving overloads or bumping the major version.
FASTNOISE_API void* fnNewFromEncodedNodeTree( const char* encodedString, unsigned /*FastSIMD::FeatureSet*/ simdLevel /*~0u = Auto*/ );
Co-authored-by: Copilot <[email protected]>
…to NewFastSIMD
…g existing strings, stack node end markers to reduce string length
…enamed terrace variables
…eturning inf on 0
ToDo