Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • VTK-m VTK-m
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 192
    • Issues 192
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 35
    • Merge requests 35
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • VTK
  • VTK-mVTK-m
  • Merge requests
  • !2686

Add `CreateResult` to `NewFilter` and absorb field mapping

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Kenneth Moreland requested to merge kmorel/vtk-m:create-result-does-field-map into master Jan 26, 2022
  • Overview 28
  • Commits 1
  • Pipelines 12
  • Changes 22

The original version of Filter classes had a helper header file named CreateResult.h that had several forms of a CreateResult function that helped correctly create the DataSet to be returned from a filter's DoExecute. With the move to the NewFilter structure, these functions did not line up very well with how DataSets should actually be created.

A replacement for these functions have been added as protected helper methods to NewFilter and NewFilterField. In addition to moving them into the filter themselves, the behavior of CreateResult has been merged with the map field to output functionality. The original implementation of Filter did this mapping internally in a different step. The first design of NewFilter required the filter implementer to call a MapFieldsOntoOutput themselves. This new implementation wraps the functionality of CreateResult and MapFieldsOntoOutput together so that the DataSet will be created correctly with a single call to CreateResult. This makes it easier to correctly create the output.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: create-result-does-field-map