Skip to content

Fix pixie plugin issues - #21100

Draft
markcmiller86 wants to merge 3 commits into
developfrom
bug-mcm86-06aug26-pixie-decomp-overlap
Draft

Fix pixie plugin issues#21100
markcmiller86 wants to merge 3 commits into
developfrom
bug-mcm86-06aug26-pixie-decomp-overlap

Conversation

@markcmiller86

Copy link
Copy Markdown
Member

Description

This resolves some of the issues reported in some of the comments of #20982

  • Fixes parallel decomposition logic to eliminate overlap between MPI ranks
  • Ensures node-centered and zone-centered variables are defined correctly
  • Replaces custom parallel decomp logic with utilities available in avtDatabase

Type of change

  • Bug fix~~
  • New feature~~
  • Documentation update~~
  • Other~~

How Has This Been Tested?

Reminders:

  • Please follow the style guidelines of this project.
  • Please perform a self-review of your code before submitting a PR and asking others to review it.
  • Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

  • I have commented my code where applicable.~~
  • I have updated the release notes.~~
  • I have made corresponding changes to the documentation.~~
  • I have added debugging support to my changes.~~
  • I have added tests that prove my fix is effective or that my feature works.~~
  • I have confirmed new and existing unit tests pass locally with my changes.~~
  • I have added new baselines for any new tests to the repo.~~
  • I have NOT made any changes to protocol or public interfaces in an RC branch.~~

@markcmiller86
markcmiller86 requested a review from biagas August 8, 2026 00:54

@biagas biagas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are there any regression tests that will be affected by this change?
Is there new sample data for creating a test, or can the problem being addressed be demonstrated by current sample data?

}
}

void

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do the new methods warrant method comment blocks?

@biagas

biagas commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

@markcmiller86 looks like the scope of this PR has expanded beyond PIXIE specific fixes.
Should the notes/title be reworked, or should the HDF5 changes in other readers be put in a new PR ?

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