Skip to content

Conversation

@olakusiak
Copy link
Collaborator

currently all the files on the new SO:UK cluster are in my private path, waiting for them to create a dedicated SOlensing storage

@olakusiak olakusiak requested a review from zatkins2 May 8, 2025 11:22
Copy link
Collaborator

@zatkins2 zatkins2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a couple small questions. But I think we probably want to wait til you move everything to their "final spot" on souk before merging into main.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so_lat_mbs_mss0002: so_lat_mbs_mss0002.yaml is a duplicate of default_config right? Could that line be deleted and you just use the default?

Same thing with lensing_masks: so_lat_mbs_mss0002.yaml?

I guess the point is you'd rather call it something else when you fetch it in code?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you rename the split_map_file_template files everywhere, like, also on perlmutter and rusty? Otherwise the files will have different names on those two vs. dirac and souk. Is it not possible to stick with the pre-existing filenames on dirac and souk?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants