GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — [SimWF,WIP] Enable MaterialManager input from CCDB by benedikt-voelkel · Pull Request #1650 · AliceO2Group/O2DPG

Conversation

benedikt-voelkel commented May 24, 2024

Copy link

Copy Markdown

Contributor

first demonstrator

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

benedikt-voelkel requested review from chiarazampolli and sawenzel as code owners

May 24, 2024 14:39

github-actions Bot commented May 24, 2024

Copy link

Copy Markdown

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2022-pp-apass4
async-2023-pbpb-apass3
async-2023-pbpb-apass4
async-2022-pp-apass6-2023-PbPb-apass2
async-2022-pp-apass4-accepted
async-2022-pp-apass6-2023-PbPb-apass2-accepted
async-2023-pbpb-apass3-accepted
async-2023-pbpb-apass4-accepted
async-2023-pp-apass4
async-2023-pp-apass4-accepted
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-pp-cpass0-accepted
async-2022-pp-apass7-accepted
async-2024-pp-apass1-accepted

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

benedikt-voelkel force-pushed the fetch-medium-params branch from 98686bd to cf6f273 Compare

June 20, 2024 09:16


          [SimWF,WIP] Enable MaterialManager input from CCDB

cc66dac

* first demonstrator

benedikt-voelkel force-pushed the fetch-medium-params branch from cf6f273 to cc66dac Compare

June 20, 2024 09:19

benedikt-voelkel commented Jun 25, 2024

Copy link

Copy Markdown

Contributor Author

@sawenzel this works fine for me.

Next should be uploading a material parameter file to the production CCDB and then change this demonstrator accordingly to use the correct paths.

Once done, I would say this can be merged.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Reviewers

sawenzel

Awaiting requested review from sawenzel

sawenzel is a code owner

chiarazampolli

Awaiting requested review from chiarazampolli

At least 0 approving reviews are required to merge this pull request.

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.