acts-project / acts-project/acts

Inconsistent use of `VolumeMaterialInteraction` thickness

Offen
#2,279 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Stale
Vorherrschende Sprache
C++
Sterne
131
Forks
276
Ø Merge
3 T. 13 Std.
Gemergte PRs (30 T.)
112

Beschreibung

`VolumeMaterialInteraction` seems to be used in two cases: material recording and dense environment energy loss.

In the material recording case we expect the thickness to be set to `1` because we later rescale it to the step length.

But the dense environment expects a properly scaled material slab which is directly used for the energy loss.

We should decide for one of the solutions. IMO the prescaled one is the less confusing one

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.