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.

Sign up for GitHub

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

Jump to bottom

Replace the pre-nn and pre-nn-edges by the MLPEncoder #234

Open
DomInvivo opened this issue Feb 27, 2023 · 1 comment
Open

Replace the pre-nn and pre-nn-edges by the MLPEncoder #234

DomInvivo opened this issue Feb 27, 2023 · 1 comment
Assignees
@shenyangHuang
Labels
enhancement New feature or request
Milestone
Graphium 3.0

Comments

@DomInvivo
Copy link
Collaborator

Replace the pre-nn and pre-nn-edges by the MLPEncoder. Also allow pre-nn-graph and pre-nn-atten (I think this would come naturally).

In MLPEncoder, replace the use of the MLP class by the FeedForwardNN class for more options.
@DomInvivo DomInvivo added this to the Documentation + refactoring milestone Feb 27, 2023
@DomInvivo DomInvivo added the enhancement New feature or request label Aug 29, 2023
@DomInvivo DomInvivo mentioned this issue Jul 9, 2024
Closed
@DomInvivo
Copy link
Collaborator Author

See issue #518

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shenyangHuang shenyangHuang

Labels
enhancement New feature or request
Projects
None yet
Milestone
Graphium 3.0
Development

No branches or pull requests
2 participants
@shenyangHuang @DomInvivo