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

Longitudinal preprocessor multithreading #373

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andro2157
Copy link
Contributor

Multithreading for Longitudinal Features Lagger

@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from cbd931a to c9818f3 Compare June 22, 2019 13:50
@PhilipDeegan PhilipDeegan requested a review from MaryanMorel June 22, 2019 13:50
@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch 3 times, most recently from c3313ae to e61301f Compare June 24, 2019 13:34
Copy link
Member

@MaryanMorel MaryanMorel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me! It'd be nice to put CooMatrix declaration in a separate file from the preprocessor though.

@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from 25136c2 to 4ae90a7 Compare July 6, 2019 16:10
@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from 4ae90a7 to 45ff515 Compare July 16, 2019 13:13
@PhilipDeegan
Copy link
Member

Should probably have a test of some sort to validate they output the same things

@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from 45ff515 to 1175859 Compare September 3, 2019 12:17
This was referenced Sep 4, 2019
@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from 1175859 to c474299 Compare October 31, 2019 19:59
uint to size_t

oops

updated swig

updates for single swig header include directory

template init

move coo_matrix to array headers
@PhilipDeegan PhilipDeegan force-pushed the longitudinal-preprocessor-parallelization branch from c474299 to f8a713d Compare May 30, 2020 15:41
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