search

NSLS-II-BMM / profile_collection

BlueSky configuration for NSLS-II beamline 6BM, BMM
https://wiki-nsls2.bnl.gov/beamline6BM/index.php/Main_Page
Other
3 stars 7 forks source link

Unreal #23

Open mguercio626 opened 1 year ago

mguercio626 commented 1 year ago

Unreal engine edits.

bruceravel commented 1 year ago

@mguercio626, @gwbischof, I am thinking that I would prefer to use an environment variable -- say $BLUESKY_UNREAL -- and use os.getenv() to fetch its value. That seems more maintainable and usable than a hard-coded switch (which is repeated several times in different places).

gwbischof commented 1 year ago

@mguercio626, @gwbischof, I am thinking that I would prefer to use an environment variable -- say $BLUESKY_UNREAL -- and use os.getenv() to fetch its value. That seems more maintainable and usable than a hard-coded switch (which is repeated several times in different places).

Sounds good, that's what we were thinking too.

gwbischof commented 1 year ago

We made a bluesky-unreal package here: https://github.com/gwbischof/bluesky-unreal/tree/main/bluesky_unreal