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

Minimax and distribution_explorer to examples. #620

Merged
merged 2 commits into from
May 3, 2021
Merged

Conversation

NAThompson
Copy link
Collaborator

@NAThompson NAThompson commented Apr 30, 2021

And while we're at it: What do these do? I've never seen these run or tested . . .

@jzmaddock
Copy link
Collaborator

The Remez code is documented here: https://www.boost.org/doc/libs/1_42_0/libs/math/doc/sf_and_dist/html/math_toolkit/toolkit/internals2/minimax.html

I'd rather not loose it, since it contains the record of how the minimax approximations were derived (and is the starting point for new ones). It could be moved under tools/minimax if we want to tidy up the root directory.

The distribution-explorer code is a .NET application that @pabristow wrote to illustrate the behaviour of our distribution classes. I confess I'm unable to find any docs on it - Paul?

@pabristow
Copy link
Contributor

@jzmaddock - I deny writing it! (My eyes are dim, I cannot see ... ... ) But I may have influenced its production. I have no reason to believe that it no longer works and it may still be a useful guide to anyone producing C# / .NET code that uses Boost.Math. Was one original purpose to show that this was possible?

Docs are at distexplorer.qbk and don't look too out of date.

It's not hurting anyone?

@NAThompson
Copy link
Collaborator Author

It could be moved under tools/minimax if we want to tidy up the root directory.

Done.

It's not hurting anyone?

Just moved to the examples directory-not deleted.

@NAThompson NAThompson merged commit 0cd6039 into develop May 3, 2021
@NAThompson NAThompson deleted the mv_to_examples branch May 3, 2021 13:13
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.

3 participants