Skip to content

Remove Open Babel support - #151

Merged
RMeli merged 3 commits into
developfrom
noob
Dec 20, 2025
Merged

Remove Open Babel support#151
RMeli merged 3 commits into
developfrom
noob

Conversation

@RMeli

@RMeli RMeli commented Dec 20, 2025

Copy link
Copy Markdown
Owner

Description

Close #149. Open Babel seems to be abandoned, especially from the Python side.

Checklist

  • Tests
  • Documentation
  • Changelog

@RMeli RMeli self-assigned this Dec 20, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes support for Open Babel from the spyrmsd library, focusing exclusively on RDKit for molecule I/O operations. The change is motivated by Open Babel being abandoned, particularly on the Python side.

  • Removed the entire spyrmsd/optional/obabel.py module
  • Updated import logic to only use RDKit for molecular operations
  • Removed the molformat() utility function that was specific to Open Babel

Reviewed changes

Copilot reviewed 39 out of 39 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
spyrmsd/optional/obabel.py Deleted entire Open Babel integration module
spyrmsd/io.py Simplified import logic to only try RDKit, removed nested fallback
spyrmsd/utils.py Removed molformat() function used for Open Babel format conversion
spyrmsd/optional/rdkit.py Changed from utils.molformat() to utils.format(), updated docstring reference
spyrmsd/molecule.py Removed from_obabel() class method, updated docstring
spyrmsd/__main__.py Updated error message to only mention RDKit requirement
setup.py Removed openbabel from extras_require
tests/test_utils.py Removed test_molformat() test function
tests/test_molecule.py Removed test_from_obmol() test function
docs/source/tutorials/tutorial.rst Updated to remove Open Babel references, changed section headings
docs/source/tutorials/tutorial.ipynb Updated notebook to remove Open Babel references
docs/source/installation.rst Removed Open Babel from installation instructions
docs/source/api/*.rst Standardized order of autodoc directives, removed obabel module docs
devtools/conda-envs/*.yaml Removed Open Babel test environments and dependency
.github/workflows/pytest.yml Removed Open Babel from test matrix
.github/workflows/pre-commit.yml Updated Python version matrix
README.md Updated to remove Open Babel references
CHANGELOG.md Added entry for Open Babel removal

馃挕 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/source/tutorials/tutorial.rst
Comment thread README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@RMeli
RMeli merged commit db19a30 into develop Dec 20, 2025
24 checks passed
@RMeli
RMeli deleted the noob branch December 20, 2025 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Open Babel support?

2 participants