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

Processor Interface #10073

Closed
wants to merge 13 commits into from
Closed

Processor Interface #10073

wants to merge 13 commits into from

Conversation

nvidianz
Copy link

Processor interface with the dummy implementation.

@hcho3
Copy link
Collaborator

hcho3 commented Feb 23, 2024

Is this related to #9987?

@ZiyueXu77
Copy link

Is this related to #9987?

yes, I will update 9987 with the HE interface design and link this PR there

@trivialfis
Copy link
Member

Hi, is this ready for review?

@nvidianz nvidianz marked this pull request as draft March 5, 2024 18:40
@nvidianz
Copy link
Author

nvidianz commented Mar 5, 2024

Hi, is this ready for review?

We are still making changes to support Secure XGBoost. I just turned it into draft. Thanks!

@nvidianz nvidianz marked this pull request as ready for review March 23, 2024 15:07
@da-niao-dan
Copy link

hello, I am just checking if I interpret this correctly:

  1. The plugin package can only be dynamically linked (by .so file etc) and not statically linked.
  2. The plugin package should create a processor class and then provide a .so file.

right?

@nvidianz
Copy link
Author

This PR is merged into #10124

@nvidianz nvidianz closed this Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants