Skip to content
Snippets Groups Projects

Draft: scifi api completeness

Closed Zehua Xu requested to merge api_completeness into master
1 unresolved thread

To fix this issue: #33 (closed), used for geometry navigation

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
  • Zehua Xu mentioned in issue #33 (closed)

    mentioned in issue #33 (closed)

    • By doing this, you expose the containers that were hidden so far behind the API. What's the usage you have in mind here ? I would rather keep the container hidden and offer the needed API to act over it, namely applyToAllChidren.

    • Author Maintainer

      Thanks a lot for your comments. This kind of functionality is there for the DetDesc version (#33 (closed)). From the local test, we can also use applyToAllChidren() to do a detector element navigation. Maybe I can close this issue(#33 (closed)).

      auto func = [this, &i]( const DeFTStation& station ) {
               std::cout << "hi" << std::endl;
        };
        deFT.applyToAllChildren( func );
      Edited by Zehua Xu
    • I would rather keep containers hidden and implement possibly missing applyToAllXXX

    • Please register or sign in to reply
  • Zehua Xu marked this merge request as draft

    marked this merge request as draft

  • closed

Please register or sign in to reply
Loading