Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • Nektar Nektar
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 93
    • Issues 93
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 59
    • Merge requests 59
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Nektar
  • NektarNektar
  • Merge requests
  • !762

Fix some issues with revised Mesh::MakeOrder routine to address failing tests

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Dave Moxey requested to merge dmoxey/nektar:fix/makeorder into master Apr 03, 2017
  • Overview 4
  • Commits 2
  • Changes 4

This MR fixes an issue with revised MakeOrder routine introduced in !747 (merged), where the edge copies used were inconsistent with the face copies, leading to failed tests where MakeOrder is used. This also fixes some issues with edge and face constructors where CAD information may not be copied properly if the copy constructor is used.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix/makeorder