Skip to content

Easier TMUX-like environment setup - #67

Merged
mroda88 merged 4 commits into
developfrom
hwallace/easier_env_setup
Jan 16, 2026
Merged

Easier TMUX-like environment setup#67
mroda88 merged 4 commits into
developfrom
hwallace/easier_env_setup

Conversation

@henry-wallace-phys

@henry-wallace-phys henry-wallace-phys commented Jan 16, 2026

Copy link
Copy Markdown
Collaborator

Description

Updates pyproject.toml to include runconf_np0X_env_setup.sh (X=2/4) which can then be sourced from anywhere (.

See issue #66 for details
Addresses issue #66

Testing done by running
source runconf_np02_env_setup.sh or source runconf_np04_env_setup.sh and

  1. Checking all variables are correct i.e.
APPARATUS="np02"


SESSION_FILE="np02-session.data.xml"
SESSION_NAME="np02-session"
CONFIG_DIR="${DBT_AREA_ROOT}/np02-runs"
OPERATION_URL="https://gitlab.cern.ch/dune-daq/online/np02-configs-operation.git"
  1. Checking runconf-ui runs correctly

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature or enhancement (non-breaking change which adds functionality)
  • Optimization (non-breaking change that improves code/performance)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Testing checklist

  • Unit tests pass (e.g. dbt-build --unittest)
  • Minimal system quicktest passes (pytest -s minimal_system_quick_test.py)
  • Full set of integration tests pass (daqsystemtest_integtest_bundle.sh)
  • Python tests pass if applicable (e.g. python -m pytest)
  • Pre-commit hooks run successfully if applicable (e.g. pre-commit run --all-files)

Comments here on the testing

Further checks

  • Code is commented where needed, particularly in hard-to-understand areas
  • Code style is correct (dbt-build --lint, and/or see https://dune-daq-sw.readthedocs.io/en/latest/packages/styleguide/)
  • If applicable, new tests have been added or an issue has been opened to tackle that in the future.
    (Indicate issue here: # (issue))

@MRiganSUSX

MRiganSUSX commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

@henry-wallace-phys @mroda88
I can confirm that this works as expected now, ie after installing runconf-ui, one can then do source runconf_np02_env_setup.sh and it sets all variables as expected, enabling to run runconf-shifter-ui without the need for either ehn1-daqconfigs or runconf-ui.

I do have 2 follow up questions:

  • is there a need for whole source, ie do we really need to so source runconf_np02_env_setup.sh instead of just runconf_np02_env_setup.sh
  • since the scripts are getting renamed, are they referenced anywhere else (especially thinking of documentation, twiki etc) where this needs to be reflected ?

@henry-wallace-phys

Copy link
Copy Markdown
Collaborator Author

So for the first comment, yes. This just adds the bash script to the bin so it's arbitrarily run-able. You can try testing with runconf_np02_env_setup.sh but (at least for me) this didn't set up the environment

Yes this setup is mentioned https://github.com/DUNE-DAQ/runconf-ui/blob/ef141aeaad32b9b21b4c84df516a5cc9fa736cff/docs/runconf_ui_config.md here

@mroda88

mroda88 commented Jan 16, 2026

Copy link
Copy Markdown
Collaborator

Also, the scripts were not used anywhere else before. they were for Henry's private setup. We renamed them because once they are for everyone it would be unclear where they would come from. Please @MRiganSUSX approve if you are ok with this so we can merge.

@MRiganSUSX
MRiganSUSX self-requested a review January 16, 2026 15:15

@MRiganSUSX MRiganSUSX left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@mroda88
mroda88 merged commit 47393d6 into develop Jan 16, 2026
2 checks passed
@mroda88
mroda88 deleted the hwallace/easier_env_setup branch January 16, 2026 15:18
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.

4 participants