Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DFP: Exclude unwanted columns #583

Merged
71 commits merged into from
Jan 21, 2023

Conversation

dagardner-nv
Copy link
Contributor

@dagardner-nv dagardner-nv commented Dec 21, 2022

  • Filter stage now optionally able to use a DataFrame column as the filter criteria.
  • Choice of a DataFrame column or a Tensor is specified with a FilterSource enum.
  • Accepted message type of the Filter stage is now driven by the choice of a DF column or Tensor as the criteria.
  • DType now has a default constructor.
  • DFP Post Processing stage no longer performs it's own filtering and no longer converts the output message to a UserMessageMeta.
  • DFP inference pipelines updated to use Filter & Serialize stages, resulting in unwanted columns excluded from output (fixes [FEA]: DFP should use a serialize stage, and exclude internal fields #582).
  • Update DFP docs to reflect these changes

Breaking Changes:

  • morpheus._lib.file_types merged into morpheus._lib.common
  • DevMemInfo expanded to include shape & stride attributes
  • MatxUtil updated to take advantage of new attrs in DevMemInfo

@dagardner-nv dagardner-nv added non-breaking Non-breaking change improvement Improvement to existing functionality 2 - In Progress labels Dec 21, 2022
@dagardner-nv dagardner-nv requested a review from a team as a code owner December 21, 2022 01:08
@dagardner-nv dagardner-nv added breaking Breaking change and removed non-breaking Non-breaking change labels Jan 17, 2023
Copy link
Contributor

@mdemoret-nv mdemoret-nv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks better using the DevMemInfo everywhere.

@mdemoret-nv
Copy link
Contributor

/merge

@ghost ghost merged commit 7e4e774 into nv-morpheus:branch-23.01 Jan 21, 2023
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking change improvement Improvement to existing functionality
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[FEA]: DFP should use a serialize stage, and exclude internal fields
2 participants