Skip to content

Update Kripke Documentation#18

Merged
pearce8 merged 3 commits intodevelopfrom
task/chen59/kripketext
Jan 23, 2026
Merged

Update Kripke Documentation#18
pearce8 merged 3 commits intodevelopfrom
task/chen59/kripketext

Conversation

@rchen20
Copy link
Member

@rchen20 rchen20 commented Dec 16, 2025

No description provided.

Most Sn transport codes are designed around one of these nestings, which is an inflexibility that leads to software engineering compromises when porting to new architectures and programming paradigms.

Early research has found that the problem dimensions (zones, groups, directions, scattering order) and the scaling (number of threads and MPI tasks), can make a profound difference in the performance of each of these nestings.
To our knowledge, this is a capability unique to Kripke, and should provide key insight into how data-layout affects Sn solver performance.
Copy link
Collaborator

Choose a reason for hiding this comment

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

@rchen20 We need details in the code modification section on what may be changed, and what should stay the same. If I am not mistaken, the data ordering (GDZ) needs to stay the same,. while the execution policy can be changed and optimizations like loop tiling are allowed.

@rchen20 rchen20 changed the title DRAFT: Update Kripke Documentation Update Kripke Documentation Jan 20, 2026
@rchen20
Copy link
Member Author

rchen20 commented Jan 20, 2026

@pearce8 We can merge this after John Loffeld and Tom take quick look.

@pearce8 pearce8 requested a review from brunner6 January 22, 2026 01:52
@pearce8
Copy link
Collaborator

pearce8 commented Jan 23, 2026

@rchen20 I was able to tag one of them here for the review - please check in with them and let me know when I should merge, in case they don't end up putting a formal review of the PR in.

@rchen20
Copy link
Member Author

rchen20 commented Jan 23, 2026

@rchen20 I was able to tag one of them here for the review - please check in with them and let me know when I should merge, in case they don't end up putting a formal review of the PR in.

John Loffeld already took a look at this and told me this was good to go. Feel free to merge it whenever you'd like.

@pearce8 pearce8 merged commit db64df6 into develop Jan 23, 2026
2 checks passed
@pearce8 pearce8 deleted the task/chen59/kripketext branch January 23, 2026 04:09
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