-
Notifications
You must be signed in to change notification settings - Fork 4
Description
At the moment, cherab.mastu.machine.MASTU_WALL_OUTLINE is a hard-coded array of values, with no indication of where these have come from or what date range they are valid for. The array is similar (but not necessarily identical) to "wall outlines" I've seen in other MAST-U codes.
It would be better to take this sort of information from an "official" source instead (e.g. UDA). However, UDA doesn't currently seem to have an exact analogue of this outline in the machine description. The closest seems to be /limiter/efit, but that differs in a couple of respects:
- It cuts off a corner between T5 and the baffle
- In the main chamber, the outer extent of the outline is at R=1.521m, rather than 2.0m.
I'm a bit worried that these differences are significant enough to break existing users' codes. There are a couple of alternatives that I can think of:
- Use
/passive/efitinstead of/limiter/efit, and extract coordinates from the relevant passive structures to match as closely as possible the existing outline. This seems like a lot of extra complexity. - Ask for the cherab.mastu wall outline to be included in the machine description too. However, that risks having unnecessary duplication in the machine description: which of the two outlines should other users of UDA use?
- Document in the code where the wall outline actually came from (as long as the source is stored properly, e.g. in Sharepoint), but leave it hard-coded in Cherab.
Of these alternatives, I personally prefer option 3. Overall, it's a toss-up between using /limiter/efit or option 3 for me.
Thoughts?