Skip to content
Snippets Groups Projects

Improve GNN4ITk pipeline

Merged Xiangyang Ju requested to merge xju/athena:mr_walkthrough into main

Add new functionality to ExaTrkX utilities:

  • add namespace for all ExaTrkX utility functions,
  • improve the brutal-force search for fixed radius nearest neighbors (buildEdges),
  • add the CCandWalk algorithm that does the connected component and walk through algorithms for graph segmentation,

Improve the TrackFinderTool based on the Metric-Learning-based GNN4ITk pipeline.

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
  • Xiangyang Ju added 2 commits

    added 2 commits

    Compare with previous version

  • Xiangyang Ju resolved all threads

    resolved all threads

  • This merge request affects 1 package:

    • InnerDetector/InDetGNNTracking

    This merge request affects 8 files:

    • InnerDetector/InDetGNNTracking/CMakeLists.txt
    • InnerDetector/InDetGNNTracking/README.md
    • InnerDetector/InDetGNNTracking/python/InDetGNNTrackingConfig.py
    • InnerDetector/InDetGNNTracking/python/InDetGNNTrackingConfigFlags.py
    • InnerDetector/InDetGNNTracking/src/ExaTrkXUtils.cxx
    • InnerDetector/InDetGNNTracking/src/ExaTrkXUtils.hpp
    • InnerDetector/InDetGNNTracking/src/SiGNNTrackFinderTool.cxx
    • InnerDetector/InDetGNNTracking/src/SiGNNTrackFinderTool.h

    Adding @goetz ,@sroe as watchers

  • CI Result SUCCESS (hash f6f8f67d)

    Athena
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output (remote access info)

  • Alexis Vallier resolved all threads

    resolved all threads

  • Changes look fine to me. Sending on to L2 for a second check. Clara (L1)

  • Looks good. Karolos (L2)

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