forked from sloisel/numeric
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Find out maintained repository #2
Open
niccokunzmann
wants to merge
69
commits into
orangeduck:master
Choose a base branch
from
GerHobbelt:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Conflicts: tools/unit2.js workshop.php
a javascript implementation of Newton solver, which is ported from GSL (GNU Scientific Library) 's globally convergent Newton method (gsl-1.15\multiroots\gnewton.c) This Newton solver has been successfully used to implement a simple geometric constraint solver in project Rena (https://github.com/kaige/Rena)
This replaces calls to javascript's Function() with calls to the new numeric.compile(), which is a wrapper that closures in the numeric object so that methods on it can be used inside compiled code. The motivation here is to allow using numeric without globally exporting the `numeric` symbol.
2. bug fixed in the benchmark page
the performance of SOR solver is not as good as CG and BiCGSTAB, but similar to ccsLUPSolve. in practice, CG and BiCGSTAB should always be the best options.
2. test added, doc updated
This avoids relying on automatic semicolon insertion, to make the code cleaner and avoid pitfalls. The svd module in particular had many unterminated lines, likely due to its Python heritage. The insertion places were identified by JSHint.
Many places in numeric.js create new functions using the Function(...) constructor. Such functions do not have access to the local scope. Therefore they bind to the global "numeric" object, which might not exist, or might belong to a different version of numeric.js, or might be something different altogether. Since we are careful enough to have a local variable in most places, it appears prudent to make sure that we pass this to functions created from source strings as well.
Due to how prettyPrint was implemented, 100 would always be rendered as 1e2. Fixed that particular error. Still leaves open the issue of rendering 10000 as 1e4, for example.
This warning was being logged to the console whenever there was a doubled semicolon after a return statement, because there is an unreachable (empty) statement between the semicolons. Regularize the usage of semicolons in generated code to avoid producing these warnings.
# Conflicts: # src/numeric.js
# Conflicts: # src/numeric.js # src/svd.js
# Conflicts: # src/numeric.js
# Conflicts: # src/numeric.js
# Conflicts: # src/numeric.js # tools/build.sh
# Conflicts: # documentation.html # src/documentation.html # src/numeric.js
…gspace` and `norm2` functions.
…pment package.json file for `npm install`
…t the UMD library file (CommonJS+AMD+Browser support)
…anually from latest github repo sources, sans minification for easier debugging.
… pages (also note: iterative.js is now part of numeric-latest.js so there's no need to include it).
…pressed library dist as well.
I'm probably not the guy to maintain |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
sloisel#72
@orangeduck and @GerHobbelt you both put in a lot of work.
I am creating this pull-request to see, if we can have a single repository coming out as the maintained one, if you both are willing to join forces.