-
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
69
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 65 commits
Commits
Show all changes
69 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 1ca7fa6
dox cleanup
evaleev ff05b7c
nns_projection_weights: no third lock needed to return the result
evaleev 4732bf5
removed the no-longer-used biorthogonalization_hardcoded.ipp
evaleev ecde7eb
memoize biorthogonal coeffs by lambda to have separate cache for hard…
ABesharat 5f490ef
Merge remote-tracking branch 'refs/remotes/origin/azam/feature/hardco…
ABesharat 333f296
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 6697a4e
Merge remote-tracking branch 'refs/remotes/origin/master' into azam/f…
ABesharat 8f42dac
remove the redundant header include
ABesharat 2e171a8
move Eigen usage from public header to implementation
ABesharat 370bd30
remove the redundant header include
ABesharat e6adbab
biorthogonalization.cpp: forward declaration to avoid having Eigen in…
ABesharat 72dac7e
move hardcoded contents to biorthogonalization.cpp and hpp
ABesharat 711e460
add to header file to keep the documentation
ABesharat f8f3115
make compute-nns compact
ABesharat 9d98421
using auto instead of std::size_t
ABesharat c3a6c07
remove biorthogonalization from IR since it is too domain-specific
evaleev c8cf5b9
fix the cmake
ABesharat 6c138ef
fix nns call in eval_ta and eval_btas
ABesharat 5b64bdd
refactor SQ/eval to live in core/eval and separate backend-specific a…
evaleev 93fab50
Merge pull request #465 from ValeevGroup/evaleev/feature/conditional-…
bimalgaudel 30f76e0
fix Copilot suggestions
ABesharat 40663b4
biorthogonalization: renames/restruct
evaleev 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.
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.