Skip to content

[WIP] Add Branson Monte Carlo Kernel#683

Draft
michaelmckinsey1 wants to merge 1 commit into
developfrom
branson-kernel-photon-transport
Draft

[WIP] Add Branson Monte Carlo Kernel#683
michaelmckinsey1 wants to merge 1 commit into
developfrom
branson-kernel-photon-transport

Conversation

@michaelmckinsey1

@michaelmckinsey1 michaelmckinsey1 commented May 20, 2026

Copy link
Copy Markdown
Contributor

Summary

An intern (@RanivG) will be implementing https://github.com/lanl/branson/blob/3f538e31a51b144b6621d4665f465d8d44ee85b8/src/event_based_transport.h#L323 as a kernel in RAJAPerf.

  • This PR is a new kernel.

  • It does the following (modify list as needed):

    • Adds Apps_TRANSPORT_3DMC kernel
  • Add reference implementation to header file (see other kernels for examples).

  • Rename to MC_EVENT_PHOTON_TRANSPORT

@michaelmckinsey1 michaelmckinsey1 self-assigned this May 20, 2026
@MrBurmark

Copy link
Copy Markdown
Member

Is there a better name for this kernel?

@michaelmckinsey1

Copy link
Copy Markdown
Contributor Author

Is there a better name for this kernel?

I am open to suggestions, but it can be renamed easily.

@rhornung67

Copy link
Copy Markdown
Member

I'm fine with the name. @MrBurmark do you have a specific concern about the name?

@MrBurmark

Copy link
Copy Markdown
Member

The name is just very generic. Is the kernel any specific part of the Monty Carlo algorithm?

@rhornung67

Copy link
Copy Markdown
Member

The name is just very generic. Is the kernel any specific part of the Monty Carlo algorithm?

That's a good point. A brief scan of the code at the link @michaelmckinsey1 provided above to the Branson code, perhaps something like MC_EVENT_PHOTON_TRANSPORT, if that's not too long.

@rhornung67

Copy link
Copy Markdown
Member

@michaelmckinsey1 please add a reference implementation to the kernel header file.

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