Skip to content
Snippets Groups Projects

Add set-to-set lookups for faster AlgsExecutionStates matching

Merged Add set-to-set lookups for faster AlgsExecutionStates matching
ishapova/Gaudi:opt-algs-states into master
All threads resolved!
Merged Illya Shapoval requested to merge ishapova/Gaudi:opt-algs-states into master
All threads resolved!

This MR is likely in conflict with !670 (merged). Please merge the latter first.

Edited by Illya Shapoval

Merge request reports

Pipeline #391551 passed

Stage: build
    Stage: test

      Pipeline passed for 164c52d5 on ishapova:opt-algs-states

      Approved by
      Marco Clemencic

      Merged by Marco ClemencicMarco Clemencic 6 years ago (Jun 15, 2018 1:56pm UTC)

      Merge details

      • Changes merged into master with 6eca38de.
      • Deleted the source branch.

      Pipeline #416233 passed

      Stage: build
        Stage: test
          Stage: deploy

            Pipeline passed for 6eca38de on master

            Activity

            Filter activity
            • Approvals
            • Assignees & reviewers
            • Comments (from bots)
            • Comments (from users)
            • Commits & branches
            • Edits
            • Labels
            • Lock status
            • Mentions
            • Merge request status
            • Tracking
          • Frank Winklmeier
          • Illya Shapoval added 2 commits

            added 2 commits

            • b383c06f - make aliases to algorithm execution states homogeneous across different modules
            • ceeb92f5 - change AlgsExecutionStates method name, and add method descriptions

            Compare with previous version

          • Illya Shapoval resolved all discussions

            resolved all discussions

          • Illya Shapoval added 1 commit

            added 1 commit

            • 6b55a768 - change AlgsExecutionStates method name, and add method descriptions

            Compare with previous version

          • Illya Shapoval changed the description

            changed the description

          • Marco Clemencic assigned to @clemenci

            assigned to @clemenci

          • Marco Clemencic changed milestone to %v30r3

            changed milestone to %v30r3

          • Illya Shapoval added 25 commits

            added 25 commits

            • 6b55a768...992791e5 - 19 commits from branch gaudi:master
            • 0e2eb0f5 - add set-based state lookups to AlgsExecutionStates, and shorten method names
            • 5d073592 - replace keyword-like logical operators to symbol-like ones
            • 1a99c271 - remove unused header
            • 314e75de - use cleaner typedefs in the scheduler
            • 16f5359e - make aliases to algorithm execution states homogeneous across different modules
            • 164c52d5 - change AlgsExecutionStates method name, and add method descriptions

            Compare with previous version

            Toggle commit list
          • Illya Shapoval
            Illya Shapoval @ishapova ·
            Author Contributor

            @clemenci thanks for merging !670 (merged) (and others). This MR, as well as !689 (merged) and !690 (merged), are now rebased onto latest master and are ready for revision as well.

          • Frank Winklmeier mentioned in merge request !690 (merged)

            mentioned in merge request !690 (merged)

          • Marco Clemencic approved this merge request

            approved this merge request

          • Marco Clemencic merged

            merged

          • Marco Clemencic mentioned in commit 6eca38de

            mentioned in commit 6eca38de

            • Please register or sign in to reply