Skip to content

Add Set2Set Aggregation#6

Merged
JacksonBurns merged 4 commits intochemprop:mainfrom
MarcS18:main
May 6, 2026
Merged

Add Set2Set Aggregation#6
JacksonBurns merged 4 commits intochemprop:mainfrom
MarcS18:main

Conversation

@MarcS18
Copy link
Copy Markdown
Contributor

@MarcS18 MarcS18 commented Mar 19, 2026

Description

This PR adds a Chemprop compatible version of the Set2Set aggregation function [1]. Set2Set is an aggregation function designed to map a set of input vectors to a single, permutation-invariant graph-level or set-level embedding. It is used in molecular property prediction tasks as a readout function to aggregate node-level information, ensuring the output is invariant to the input order of atoms.

This code is tested and working with Chemprop 2.2.1

Questions

None

Checklist

  • License file included
  • Documentation provided (perhaps a Notebook or README)
  • Tests provided and passing on the stated Chemprop version

@JacksonBurns JacksonBurns self-requested a review March 26, 2026 14:16
@JacksonBurns
Copy link
Copy Markdown
Member

Hi thanks for the contrbution @MarcS18 , I will get to reviewing this!

fix equation formatting on github
Copy link
Copy Markdown
Member

@JacksonBurns JacksonBurns left a comment

Choose a reason for hiding this comment

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

Hi @MarcS18 thanks for you patience! This looks to be in great shape, nicely done figuring out all of the extra 'stuff' to get this added.

Going to add your name to the contributors and then merge.

@JacksonBurns JacksonBurns merged commit ec36daf into chemprop:main May 6, 2026
8 checks passed
@JacksonBurns JacksonBurns mentioned this pull request May 6, 2026
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.

2 participants