Skip to content

Conversation

@mpuccio
Copy link
Collaborator

@mpuccio mpuccio commented Aug 13, 2025

The changes made in this commit simplify the process of uploading OTS objects
to the CCDB. The main changes are:

  • The default value for the chunkedProcessing parameter is set to true,
    making it the default behavior.
  • The base CCDB path is updated to use the EventFiltering/Zorro/ directory
    instead of the hardcoded Users/m/mpuccio/EventFiltering/OTS/ path.
  • The uploadOTSobjects function that takes a periodName parameter has been
    updated to always use chunkedProcessing = true, simplifying the function
    call.

These changes make the OTS object upload process more consistent and easier to
use, while also ensuring that the CCDB path is more generic and not tied to a
specific user's directory.

The changes made in this commit simplify the process of uploading OTS objects
to the CCDB. The main changes are:

- The default value for the `chunkedProcessing` parameter is set to `true`,
  making it the default behavior.
- The base CCDB path is updated to use the `EventFiltering/Zorro/` directory
  instead of the hardcoded `Users/m/mpuccio/EventFiltering/OTS/` path.
- The `uploadOTSobjects` function that takes a `periodName` parameter has been
  updated to always use `chunkedProcessing = true`, simplifying the function
  call.

These changes make the OTS object upload process more consistent and easier to
use, while also ensuring that the CCDB path is more generic and not tied to a
specific user's directory.
@github-actions
Copy link

O2 linter results: ❌ 5 errors, ⚠️ 0 warnings, 🔕 0 disabled

@github-actions github-actions bot changed the title feat: Update OTS object upload process [Trigger] feat: Update OTS object upload process Aug 13, 2025
@mpuccio mpuccio enabled auto-merge (squash) August 15, 2025 08:01
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @mpuccio.

@mpuccio mpuccio merged commit ab93ffe into master Aug 15, 2025
11 of 14 checks passed
@mpuccio mpuccio deleted the mp-branch-17 branch August 15, 2025 08:01
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants