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

kmp5VT [Feature] Round Robin pmap #235

Merged
merged 33 commits into from
Jan 9, 2021
Merged

kmp5VT [Feature] Round Robin pmap #235

merged 33 commits into from
Jan 9, 2021

Conversation

kmp5VT
Copy link
Contributor

@kmp5VT kmp5VT commented Dec 21, 2020

Make a round robin pmap so t2 can match mixed contraction pmap

@kmp5VT kmp5VT requested review from asadchev and evaleev December 21, 2020 19:55
@kmp5VT
Copy link
Contributor Author

kmp5VT commented Dec 22, 2020

This is almost ready to push. I want to optimize some BTAS code. Specifically the norm column code (as it has too many division operations) and the code that generates V. I want to store \beta^T \beta (or partial grammian) matrices to reduce the per iteration cost. Should be done in the next couple of days.

@evaleev
Copy link
Member

evaleev commented Dec 28, 2020

@kmp5VT please resolve CI errors

src/TiledArray/pmap/round_robin_pmap.h Show resolved Hide resolved
src/TiledArray/pmap/round_robin_pmap.h Outdated Show resolved Hide resolved
tests/round_robin_pmap.cpp Outdated Show resolved Hide resolved
external/versions.cmake Outdated Show resolved Hide resolved
src/CMakeLists.txt Outdated Show resolved Hide resolved
src/TiledArray/conversions/vector_of_arrays.h Outdated Show resolved Hide resolved
@kmp5VT kmp5VT force-pushed the updated_round_robin branch from 8b7535b to 1834501 Compare December 29, 2020 15:32
@kmp5VT kmp5VT force-pushed the updated_round_robin branch from 1834501 to d1e5a34 Compare January 4, 2021 03:14
@kmp5VT kmp5VT changed the title Round Robin pmap kmp5VT [Feature] Round Robin pmap Jan 8, 2021
@kmp5VT kmp5VT merged commit f38f32e into master Jan 9, 2021
@evaleev evaleev deleted the updated_round_robin branch January 10, 2021 13:49
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.

2 participants