Skip to content

chore: BNL Test and Cronjob updates#51

Merged
kratsg merged 8 commits into
usatlas:mainfrom
qibinlei:bnl-cron-updates
Apr 21, 2026
Merged

chore: BNL Test and Cronjob updates#51
kratsg merged 8 commits into
usatlas:mainfrom
qibinlei:bnl-cron-updates

Conversation

@qibinlei
Copy link
Copy Markdown
Contributor

  • Changing directories to point to qlei BNL test files
  • Updating BNL cronjobs to run with new parsing scripts

Copy link
Copy Markdown
Member

@kratsg kratsg left a comment

Choose a reason for hiding this comment

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

LGTM, nice job cleaning it up a bit.

@kratsg kratsg changed the title BNL Test and Cronjob updates chore: BNL Test and Cronjob updates Apr 20, 2026
@kratsg
Copy link
Copy Markdown
Member

kratsg commented Apr 20, 2026

@qibinlei please fix the pre-commit errors and then we can merge this in.

@kratsg kratsg merged commit 1e91a20 into usatlas:main Apr 21, 2026
24 checks passed
qibinlei added a commit to qibinlei/af-benchmarking that referenced this pull request May 3, 2026
* Changes to make BNL tests run on qlei and use updated parsing script

* Remove print statements from testing

* Forgot to add columnar and standard files

* Pre-commit bot fixes: suppressing error on secrets, adding || exit, and adding quotations

* style: pre-commit fixes

* suppress fastframes error

---------

Co-authored-by: Qi Bin Lei <qlei@attsub01.usatlas.bnl.gov>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Giordon Stark <kratsg@gmail.com>
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.

2 participants