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

LU Factorisation #10 issue and Histogram #5 issue #32

Merged
merged 2 commits into from
Dec 15, 2024
Merged

Conversation

dheermrt
Copy link
Contributor

@dheermrt dheermrt commented Dec 3, 2024

Changes in LU_factorisation.cpp:-
1-Accepted size of array from user instead of manually entering it as 2
2-Instead of manually entering matrix A we receive matrix a from user so that we can work with any size matrices
3-Also freed the dynamically allocated memory after the end of the program

Changes is histo.cpp
1-Dynamic allocation of arr based on Points_per_process :- Instead of statically allocating arr[100] instead malloc is used to allocate it dynamically based on Points_per_process
2-Finding max_value dynamically using std::max_element- Used the algoritihm library to calculate the maximum element in p array
3-Corrected range comparison logic for bar division-The range check was corrected to properly identify the bar a point belongs to.

4-Optimized MPI_Bcast and MPI_Scatter logic- The second version broadcasts the Range variable, while the first version only broadcasts the size, Bars, and Points_per_process as Range is now a variable which needs to be shared across processes

@dheermrt dheermrt changed the title Histogram #5 issue LU Factorisation #10 issue and Histogram #5 issue Dec 4, 2024
@LoPA607 LoPA607 merged commit abfb65e into wncc:main Dec 15, 2024
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