radiasoft / rslaser

Integrated modeling of CPA crystal-based laser amplifiers
Apache License 2.0
1 stars 1 forks source link

Create a dedicated class for laser heating #48

Closed morhenderson closed 1 year ago

morhenderson commented 1 year ago

Create a class that encapsulates & parameterizes laser heating calculations for the Crystal element. An instance of this class should be initialized by a Crystal to compute/load its n0/n2 values & ABCD matrix. The new class should feature:

bruhwiler commented 1 year ago

@morhenderson -- please provide an estimate for when this effort will be completed. It would be great if you could wrap it up this week.

bruhwiler commented 1 year ago

It would be a good idea to discuss with @bnash and show him the new code, because he needs to think about consistency between the thermal and amplifier tabs in the Silas app.

morhenderson commented 1 year ago

The code for the new class is all written and ready for testing, though I haven't discussed with software how it will work with the app.

I'm happy to meet with @bnash sometime this week to go over it and can get started on testing to make sure I'm not breaking anything in the examples or notebooks. There are one or two other things I'd like to discuss with @k-wolfinger as well before this goes into any kind of production.

bruhwiler commented 1 year ago

Discussions regarding the Silas app are not part of the scope and should not delay creation of a PR. When the PR is reviewed, we need to document any changes that break the API and notify the Sirepo team.

morhenderson commented 1 year ago

Sorry, I'm realizing I never gave an actual estimate. I plan to have testing done and a PR created today or tomorrow.

The comment about the app was more about my ability to assist @bnash with consistency between tabs. I met with @k-wolfinger about this and kept it in mind when designing the new class, but it will likely need some further consideration.