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

Fixing psparse matrix assembly col partition #115

Merged
merged 12 commits into from
Aug 15, 2023

Conversation

amartinhuertas
Copy link
Member

WIP ... please do NOT merge yet. Aims at fixing #111

This PR CAN ONLY BE reviewed/merged after PR #114

assemble_coo_with_column_owner!(...) which also sends to the
process owner of a given row the column owners of the entries
in such a row. This is required to solve the current
limitation in the algorithm which assembles the parallel sparse
matrix with the SubAssembledRows() strategy:

  The global column identifiers of matrix entries located in rows
  that a given processor owns have to be such they belong to the set of
  ghost DoFs in the local partition of the FE space corresponding
  to such processor.
…tributed.jl into fixing_psparse_matrix_assembly_col_partition
…tributed.jl into fixing_psparse_matrix_assembly_col_partition
@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2023

Codecov Report

Merging #115 (a23242d) into master (100f959) will not change coverage.
The diff coverage is 0.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master    #115    +/-   ##
=======================================
  Coverage    0.00%   0.00%            
=======================================
  Files          11      11            
  Lines        1982    2082   +100     
=======================================
- Misses       1982    2082   +100     
Files Changed Coverage Δ
src/Algebra.jl 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

…fixing_psparse_matrix_assembly_col_partition

Conflicts:
Project.toml
…tributed.jl into fixing_psparse_matrix_assembly_col_partition
…tributed.jl into fixing_psparse_matrix_assembly_col_partition
@amartinhuertas amartinhuertas changed the base branch from master to partitioned_arrays_v0.3 August 11, 2023 01:45
…tributed.jl into fixing_psparse_matrix_assembly_col_partition
…fixing_psparse_matrix_assembly_col_partition
@amartinhuertas amartinhuertas changed the base branch from partitioned_arrays_v0.3 to master August 15, 2023 06:59
@amartinhuertas amartinhuertas marked this pull request as ready for review August 15, 2023 07:09
@amartinhuertas amartinhuertas merged commit 2e0b138 into master Aug 15, 2023
6 checks passed
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.

None yet

2 participants