Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • E escet
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare
    • Locked Files
  • Issues 93
    • Issues 93
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Releases
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Eclipse ProjectsEclipse Projects
  • Eclipse ESCET (Supervisory Control Engineering Toolkit)
  • escet
  • Merge requests
  • !457

#378 CIF data-based synthesis variable orders in separate classes

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Dennis Hendriks requested to merge 378-cif-datasynth-orders-in-separate-classes into develop Dec 29, 2022
  • Overview 0
  • Commits 7
  • Pipelines 1
  • Changes 24

This is another step in #378 (closed):

  • It mainly separates the variable orders, e.g., model/random/custom order, from the CIF to synthesis converter into separate classes, that allows them to be used multiple times. They are however not yet used multiple times. That will come later in #378 (closed).
  • Moved and renamed some things, to make it all fit better. Reviewing separate commits is probably easier.
  • Currently, a VarOrderHelper is created twice, once for the VarOrder and once for VarOrderer. This is temporary. It keeps the changes smaller for this merge request. It will be changed again in a later step. (Making progress while keeping everything backwards compatible after each step is tricky.)
  • Another small change in CIF to synthesis converter: two variables named variables and curOrder had the same value, so one of them was removed.

Addresses #378 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 378-cif-datasynth-orders-in-separate-classes

Copyright © Eclipse Foundation, Inc. All Rights Reserved.     Privacy Policy | Terms of Use | Copyright Agent