Skip to content

Add bayer macrocycle datasets#120

Open
hannahbaumann wants to merge 8 commits into
mainfrom
add_bayer_macrocycles
Open

Add bayer macrocycle datasets#120
hannahbaumann wants to merge 8 commits into
mainfrom
add_bayer_macrocycles

Conversation

@hannahbaumann

Copy link
Copy Markdown
Contributor

No description provided.

@hannahbaumann hannahbaumann requested a review from jthorton April 15, 2026 12:29
Comment thread openfe_benchmarks/data/data_generation/generate_lomap_networks.py Outdated
Comment on lines +7 to +8
brd4: ["protein","bfe"]
ftase: ["protein", "cofactor", "bfe"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should this have the ross tags as well or are they just for the sets we ran as part of the industry benchmarks?

@hannahbaumann hannahbaumann Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

have two tags, ross and industry_benchmarks

Comment on lines +92 to +94
off_mols = toolkit.Molecule.from_file(
input_path.as_posix(), allow_undefined_stereo=True
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If this generally works better, can we reduce the number of scripts and switch to this method of loading the ligands cc @jaclark5 ?

@jaclark5 jaclark5 Apr 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

That's fine with me. charge_freesolv.py exists mostly because it processes several individual sdf files. I think this does pretty much the same thing as charge_mnsol.py

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.

3 participants