Skip to content

[EMCAL-630] DPL component for Raw to Cell conversion#3079

Merged
sawenzel merged 1 commit intoAliceO2Group:devfrom
hahassan7:EMCAL-630
May 14, 2020
Merged

[EMCAL-630] DPL component for Raw to Cell conversion#3079
sawenzel merged 1 commit intoAliceO2Group:devfrom
hahassan7:EMCAL-630

Conversation

@hahassan7
Copy link
Copy Markdown
Contributor

No description provided.

@hahassan7 hahassan7 requested a review from a team as a code owner March 4, 2020 17:38
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 8, 2020

This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days.

@github-actions github-actions bot added the stale label May 8, 2020
@hahassan7 hahassan7 requested a review from mfasDa as a code owner May 10, 2020 15:10
@hahassan7 hahassan7 changed the title [WIP][EMCAL-630] DPL component for Raw to Cell conversion [EMCAL-630] DPL component for Raw to Cell conversion May 10, 2020
Copy link
Copy Markdown
Collaborator

@mfasDa mfasDa left a comment

Choose a reason for hiding this comment

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

For the moment there are a couple of small modifications, however the component is not yet adapted to timeframe-based format. This needs to be done as a second step.

@hahassan7
Copy link
Copy Markdown
Contributor Author

The failing test is not due to this commit. I think the pull request can be merged.

mfasDa
mfasDa previously approved these changes May 13, 2020
@mfasDa
Copy link
Copy Markdown
Collaborator

mfasDa commented May 13, 2020

For merging the commit should be squash-merged, keeping only the commit message of the first commit.

-DPL component for Raw to Cell conversion
-Adapting the component to the new RecoWorkflow
@hahassan7
Copy link
Copy Markdown
Contributor Author

Pull request can be merged now.

@sawenzel sawenzel merged commit fe27669 into AliceO2Group:dev May 14, 2020
@hahassan7 hahassan7 deleted the EMCAL-630 branch May 14, 2020 12:56
amorsch pushed a commit to amorsch/AliceO2 that referenced this pull request May 28, 2020
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