radiasoft / sirepo

Sirepo is a framework for scientific cloud computing. Try it out!
https://sirepo.com
Apache License 2.0
64 stars 32 forks source link

Laser tab clarifications/updates #6005

Open gurhar1133 opened 1 year ago

gurhar1133 commented 1 year ago

@k-wolfinger I think wee need some more context for these Laser tab update requests

k-wolfinger commented 1 year ago

The second item, about how we initialized the function and indicate the slice positions, is not yet implemented in rslaser.

Essentially, we currently give all the laser slices the same time values and different position values, relative to the center of the pulse. We want to change that so that they all have the same position value and different time values, relative to the center of the pulse.

So, instead of plotting longitudinally vs z, we would plot longitudinally vs t. And that t would be the time for that slice to propagate to the center of the pulse (values around +40ps or -40 ps).

I hope that helps for that point?

k-wolfinger commented 1 year ago

For the first item, I think the confusion would be cleared up if we wrote something like "Number of Grid Cells in Transverse Slice" and for the two boxes: "Horizontal (x)", "Vertical (y)".

This is not a personal point of confusion, so I am not certain what will clear it up, but changes like these will help.