FESTIM icon indicating copy to clipboard operation
FESTIM copied to clipboard

A conglomerate trap as class?

Open RemDelaporteMathurin opened this issue 3 years ago • 1 comments

Something like


trap_1 = F.Trap(1, 1, 1, 1, density=1, materials=tungsten)
trap_2 = F.Trap(2, 2, 2, 2, density=2, materials=copper)


trap_conglo = F.Multitrap([trap_1, trap_2])

This would be the equivalent of:

trap_conglo = F.Trap([1, 2], [1, 2], [1, 2], [1, 2], density=[1, 2], materials=[tungsten, copper])

But more readable

RemDelaporteMathurin avatar Jun 11 '22 15:06 RemDelaporteMathurin

This would simplify the parameters logic inside FESTIM.Trap

RemDelaporteMathurin avatar Jul 02 '22 15:07 RemDelaporteMathurin