Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • D Detector
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 30
    • Issues 30
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 23
    • Merge requests 23
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • Detector
  • Merge requests
  • !310

Fix concrete material definition as sum of fractions was not 1

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ben Couturier requested to merge fix_concrete into master Oct 24, 2022
  • Overview 10
  • Commits 4
  • Pipelines 4
  • Changes 5

Fix concrete material definition as sum of fractions was 0.98 instead of 1 causing DD4hep to produce a warning. It was:

0.520 Oxygen
0.325 Silicon
0.060 Calcium
0.015 Sodium
0.020 Iron
0.040 Aluminium

This patch keeps the same fractions but normalizes them.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_concrete