You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add configurable detector version support for datasets (#11)
* Add configurable detector version support for datasets
- Add DETECTOR_CONFIG and DETECTOR_VERSION environment variables
- Set global defaults to epic_craterlake and main in .gitlab-ci.yml
- Configure eAu BeAGLE samples to use epic_craterlake_without_zdc
- Split SIDIS jobs into eAu and ep categories
- Split DIS eA jobs into eAu and other nuclei categories
- Export detector variables in determine_timing.sh before run.sh calls
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add workflow dispatch support for custom pipeline variables
- Add workflow_dispatch inputs for image_tag, detector_config, and detector_version
- Make container image tag configurable via IMAGE_TAG variable in GitLab CI
- Pass workflow inputs as variables to GitLab pipeline trigger
- Set sensible defaults for all configurable variables
This allows manual workflow triggers with custom container images
and detector configurations while maintaining backward compatibility.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add configurable container name support
- Add workflow_dispatch input for container_name (default: eic_xl)
- Add CONTAINER_NAME variable to GitLab CI (default: eic_xl)
- Update image path to use ${CONTAINER_NAME} variable
- Pass container_name from workflow to GitLab pipeline
This allows switching between different container images like
eic_xl and eic_ci via workflow dispatch.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add configurable artifact directory structure
- Add RESULTS_BASE variable combining all 4 config variables
- Update all artifact paths to use ${RESULTS_BASE}
- New structure: results/${CONTAINER_NAME}/${IMAGE_TAG}/${DETECTOR_CONFIG}/${DETECTOR_VERSION}/datasets/
This allows multiple configurations to be run in parallel with
separate artifact directories for each combination of settings.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Change default container to eic_ci
Update default CONTAINER_NAME from eic_xl to eic_ci in both
GitLab CI variables and GitHub workflow dispatch inputs.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add debug output for detector configuration variables
Add echo statements to .timings template to debug whether
DETECTOR_CONFIG and DETECTOR_VERSION are being set correctly
from job variables.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Fix RESULTS_BASE to respect job-specific variable overrides
Move RESULTS_BASE calculation from global variables to runtime
in each job template's script section. This ensures RESULTS_BASE
is calculated using the actual DETECTOR_CONFIG and other variables
after job-specific overrides are applied.
Previously, RESULTS_BASE was evaluated once globally with the
default DETECTOR_CONFIG value and never recalculated when jobs
overrode DETECTOR_CONFIG.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add debug output in determine_timing.sh and use RESULTS_BASE for logs
- Add debug echo statements to show DETECTOR_CONFIG, DETECTOR_VERSION, and RESULTS_BASE
- Update logfile path to use RESULTS_BASE instead of hardcoded results/
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add override support to nevents jobs and use YAML anchors for shared variables
- Add DETECTOR_CONFIG_OVERRIDE and DETECTOR_VERSION_OVERRIDE support to .nevents template
- Use YAML anchors to define variables once and reuse in both nevents and timings jobs
- Ensures both job types write to and read from the same RESULTS_BASE path
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Add NEVENTS_PER_TEST global variable with override support
Add NEVENTS_PER_TEST as a global variable (default: 100) with override
support via NEVENTS_PER_TEST_OVERRIDE. Also re-add EBEAM and PBEAM
global variables with override support that were lost during rebase.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Replace DEBUG with INFO and print all relevant variables in nevents/timings
- Add INFO logging to .nevents and .timings templates showing all variables
- Replace DEBUG with INFO in determine_timing.sh
- Remove redundant exports from determine_timing.sh (variables come from CI)
- Print EBEAM, PBEAM, and NEVENTS_PER_TEST in addition to existing variables
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
---------
Co-authored-by: Sakib Rahman <[email protected]>
Co-authored-by: Claude <[email protected]>
0 commit comments