Skip to content

#212 - Multi-threaded NoiseMapBuilder with Rayon [WIP] - #213

Draft
barneydmedia wants to merge 4 commits into
Razaekel:developfrom
barneydmedia:feature/rayon
Draft

#212 - Multi-threaded NoiseMapBuilder with Rayon [WIP]#213
barneydmedia wants to merge 4 commits into
Razaekel:developfrom
barneydmedia:feature/rayon

Conversation

@barneydmedia

Copy link
Copy Markdown

No description provided.

Comment thread Cargo.toml
keywords = ["math", "random"]
authors = ["The Noise-rs Developers."]

[profile.release]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to leave this in? Enabling debug in release builds is useful for debugging and benchmarking, but it can create release binaries that are very large.

Also, I don't actually know if this has any effect for libraries.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely didn't mean to commit that, feel free to remove.

@Razaekel Razaekel changed the title #212 - Multi-threaded NoiseMapBuilder with Rayon #212 - Multi-threaded NoiseMapBuilder with Rayon [WIP] Sep 20, 2019
@Razaekel
Razaekel force-pushed the develop branch 3 times, most recently from 4157bee to 7ad5b86 Compare December 4, 2022 17:34
@Razaekel
Razaekel marked this pull request as draft August 18, 2023 03:48
@abey79

abey79 commented Oct 8, 2023

Copy link
Copy Markdown

Does this have a chance to land? It's exactly what I was looking for and it looks like a really nice improvement.

@PrinceOfBorgo

Copy link
Copy Markdown
Contributor

Is this PR still in active development?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants