Skip to content
Snippets Groups Projects

GeoMaterial - Minor clean-up

Merged Johannes Junggeburth requested to merge GeoMaterialCleanUp into main

GeoElement:

  • Return the atomic number and Z by value and not const reference

GeoMaterial:

-Use the throwExcept macros

-Instead of having separate element and fraction vector, condense both into a single vector of pairs --> sort the elements inside the material by their fraction. Highest element comes first

Edited by Johannes Junggeburth

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Johannes Junggeburth resolved all threads

    resolved all threads

  • added 1 commit

    • 46810e96 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 3 commits

    • 4e4aa7ab - 1 commit from branch main
    • a8323ab2 - Clean up of the GeoMaterial
    • 7d37c592 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • OK, I rebased too early. The new Athena nightly build that contains GeoModel 6.7.0 is not yet available...

  • added 1 commit

    • 478a05fb - Revert "Apply 1 suggestion(s) to 1 file(s)"

    Compare with previous version

  • added 1 commit

    • 08a47b41 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added 1 commit

    • 172f73a6 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Vakhtang Tsulaia added 17 commits

    added 17 commits

    Compare with previous version

  • Vakhtang Tsulaia resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading