Skip to content

BUG: Modified ase.io.sdf for stable reading of SD files with >= 100 atoms

cisert requested to merge cisert/ase:fix_sdf_reading into master

When reading an Atoms object from SDF (https://wiki.fysik.dtu.dk/ase/dev/_modules/ase/io/sdf.html) with >=100 atoms, most software packages (RDKit, ChemDraw, etc.) concatenate natoms and nbonds (see attached large_molecule.sdf). This returns an error as natoms is set to the concatenated number and doesn’t find enough atom coordinates in the file. Counting the number of lines with 16 entries (lines with atom coordinates) fixes this.

Merge request reports