-
Notifications
You must be signed in to change notification settings - Fork 8
Azam/feature/hardcoded biorthogonal coefficients #457
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
Open
ABesharat
wants to merge
47
commits into
master
Choose a base branch
from
azam/feature/hardcoded_biorthogonal_coefficients
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
Changes from 35 commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
e1dba87
set hardcoded biorthogonal coeffs till rank 7
ABesharat 1323fc6
rank 7
ABesharat 897a42a
discard redundant assertion
ABesharat a22809c
better name: computed_coefficients
ABesharat dbcd4e5
add hardcoded file to cmakelist
ABesharat 8fd5be0
v2: correct rescaling factor
ABesharat 47ccfe1
renaming original coeffs to computed_coefficients
ABesharat 93829df
Merge remote-tracking branch 'origin/azam/feature/hardcoded_biorthogo…
ABesharat 70ef68a
renaming
ABesharat 4fc0898
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat fc0dcda
discard old coefficients
ABesharat f309f62
discard only the old coefficients, not num_perms
ABesharat b5da4b8
no inline anymore for performance
ABesharat 25bb6f4
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 3926bde
at max rank 6 is enough
ABesharat e485b9c
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 87b8177
test_eval_ta: add rank 4 nns test, fix rank 3
ABesharat 5ca6d55
test_eval_btas: fix nns for rank 3
ABesharat 188825c
add hardcoded biortho coeffs and nns projection
ABesharat 8e66ac7
fix include
ABesharat 1dc89b3
use hardcoded biortho coeffs for rank<=6, otherwise use computed ones
ABesharat 53b47be
do not call nns with int type ,use hardcoded nns for rank<=6, otherwi…
ABesharat a28d886
make libperm public to build the same order of permutations
ABesharat ab8f453
fix include for Eigen!
ABesharat d5b20f6
change the name of hardcoded nns fn
ABesharat 278dcc3
renaming again!
ABesharat aede8b3
renaming hardcoded_biortho_coeffs fn
ABesharat 4b1a92f
change assert to SEQUANT_ASSERT
ABesharat 8ee5a57
no need to have rescaling for v2. new nns is enough for restoring WK …
ABesharat a3345c4
documentation correction
ABesharat 78dc616
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat d2323d0
use abort for calling nns with int type
ABesharat 0f4ab0c
normalization factor of S
ABesharat acc25f8
Remove additional print statements, cleaning
ABesharat 877e3d4
address the requested changes
ABesharat 0a60656
address the requested changes, except the namespace
ABesharat 14fbd13
extract libperm calls into a helper function to avoid public dependency
ABesharat f4a0c82
Optimize build command with parallel jobs
ABesharat 7087e16
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 519e68a
a better name: biorth_threshold
ABesharat 2e67e8e
use default threshold for compute_nns_p_matricx to avoid an additiona…
ABesharat aa7a41c
add some documentations
ABesharat 69ab3a1
clear documentation
ABesharat af93394
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 2d10389
apply encapsulation for nns
ABesharat a9a17b1
memoize nns_projection weights
ABesharat afed9c4
add include
ABesharat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.