-
Notifications
You must be signed in to change notification settings - Fork 845
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Prefer wedging ring bonds around atropisomers (#7373)
* prefer wedging ring bonds for atropisomers * update expected results * handle 3D as well * prefer wedging to larger rings wedging bonds in 5 rings make later attempts at improving kekulization very difficult update expected results * extend those changes to 3D * update tests for those changes * avoid wedging atropisomers if possible response to review
- Loading branch information
1 parent
a2b149a
commit 18d1ff5
Showing
51 changed files
with
405 additions
and
92 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
Code/GraphMol/MarvinParse/test_data/JDQ443_3d.mrv.noReapply.expected.mrv
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
Code/GraphMol/MarvinParse/test_data/RP-6306_3d.mrv.noReapply.expected.mrv
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<cml xmlns="http://www.chemaxon.com" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.chemaxon.com http://www.chemaxon.com/marvin/schema/mrvSchema_20_20_0.xsd"><MDocument><MChemicalStruct><molecule molID="m1"><atomArray><atom id="a1" elementType="O" x3="-4.84493" y3="1.39393" z3="-1.47056"/><atom id="a2" elementType="O" x3="4.01651" y3="4.2266" z3="-0.04928"/><atom id="a3" elementType="N" x3="0.20832" y3="1.08593" z3="0.538347"/><atom id="a4" elementType="N" x3="0.83608" y3="-1.31367" z3="-0.582773"/><atom id="a5" elementType="N" x3="0.2296" y3="3.55497" z3="1.58667"/><atom id="a6" elementType="N" x3="4.38032" y3="3.23372" z3="2.31635"/><atom id="a7" elementType="C" x3="1.20829" y3="0.06412" z3="-0.05544"/><atom id="a8" elementType="C" x3="2.64283" y3="0.707933" z3="-0.00989333"/><atom id="a9" elementType="C" x3="-1.34773" y3="0.84196" z3="0.683013"/><atom id="a10" elementType="C" x3="0.96712" y3="2.36815" z3="0.963387"/><atom id="a11" elementType="C" x3="2.47669" y3="2.16692" z3="0.63728"/><atom id="a12" elementType="C" x3="-2.32325" y3="1.24833" z3="-0.486453"/><atom id="a13" elementType="C" x3="-1.92304" y3="0.191987" z3="1.99883"/><atom id="a14" elementType="C" x3="3.84963" y3="-0.156333" z3="-0.571013"/><atom id="a15" elementType="C" x3="3.52707" y3="-1.59273" z3="-1.1312"/><atom id="a16" elementType="C" x3="2.0412" y3="-2.09599" z3="-1.10805"/><atom id="a17" elementType="C" x3="-3.87352" y3="1.00473" z3="-0.340667"/><atom id="a18" elementType="C" x3="-3.47349" y3="-0.0516133" z3="2.14461"/><atom id="a19" elementType="C" x3="3.63907" y3="3.26471" z3="0.918587"/><atom id="a20" elementType="C" x3="-1.72536" y3="1.94759" z3="-1.90699"/><atom id="a21" elementType="C" x3="-0.91168" y3="-0.257693" z3="3.27563"/><atom id="a22" elementType="C" x3="-4.44883" y3="0.354573" z3="0.974773"/><atom id="a23" elementType="C" x3="4.77867" y3="-2.54529" z3="-1.73656"/><atom id="a24" elementType="C" x3="1.67981" y3="-3.63617" z3="-1.70688"/><atom id="a25" elementType="H" x3="4.99315" y3="0.287747" z3="-0.564293"/><atom id="a26" elementType="H" x3="-3.94165" y3="-0.556733" z3="3.16157"/><atom id="a27" elementType="H" x3="0.783253" y3="4.50865" z3="1.90493"/><atom id="a28" elementType="H" x3="-0.90216" y3="3.56319" z3="1.77035"/><atom id="a29" elementType="H" x3="-1.15715" y3="3.01831" z3="-1.65835"/><atom id="a30" elementType="H" x3="-2.6096" y3="2.20108" z3="-2.73336"/><atom id="a31" elementType="H" x3="-0.929973" y3="1.18225" z3="-2.4668"/><atom id="a32" elementType="H" x3="-0.0927733" y3="-1.10889" z3="2.90827"/><atom id="a33" elementType="H" x3="-1.5512" y3="-0.753293" z3="4.21307"/><atom id="a34" elementType="H" x3="-0.311733" y3="0.724733" z3="3.72773"/><atom id="a35" elementType="H" x3="-5.65413" y3="0.158387" z3="1.10395"/><atom id="a36" elementType="H" x3="4.88189" y3="-3.59511" z3="-1.09013"/><atom id="a37" elementType="H" x3="5.87029" y3="-1.96457" z3="-1.66413"/><atom id="a38" elementType="H" x3="4.58789" y3="-2.81241" z3="-2.92955"/><atom id="a39" elementType="H" x3="2.27547" y3="-4.50865" z3="-1.06325"/><atom id="a40" elementType="H" x3="1.98072" y3="-3.72428" z3="-2.90304"/><atom id="a41" elementType="H" x3="0.4676" y3="-3.87081" z3="-1.61261"/><atom id="a42" elementType="H" x3="4.11488" y3="2.46745" z3="3.12779"/><atom id="a43" elementType="H" x3="5.21267" y3="3.98412" z3="2.55883"/><atom id="a44" elementType="H" x3="-5.87029" y3="1.14044" z3="-1.16611"/></atomArray><bondArray><bond id="b1" atomRefs2="a1 a17" order="1"/><bond id="b2" atomRefs2="a1 a44" order="1"/><bond id="b3" atomRefs2="a2 a19" order="2"/><bond id="b4" atomRefs2="a3 a7" order="1"/><bond id="b5" atomRefs2="a3 a9" order="1"/><bond id="b6" atomRefs2="a3 a10" order="1"><bondStereo>W</bondStereo></bond><bond id="b7" atomRefs2="a4 a7" order="2"/><bond id="b8" atomRefs2="a4 a16" order="1"/><bond id="b9" atomRefs2="a5 a10" order="1"/><bond id="b10" atomRefs2="a5 a27" order="1"/><bond id="b11" atomRefs2="a5 a28" order="1"/><bond id="b12" atomRefs2="a6 a19" order="1"/><bond id="b13" atomRefs2="a6 a42" order="1"/><bond id="b14" atomRefs2="a6 a43" order="1"/><bond id="b15" atomRefs2="a7 a8" order="1"/><bond id="b16" atomRefs2="a8 a11" order="1"/><bond id="b17" atomRefs2="a8 a14" order="2"/><bond id="b18" atomRefs2="a9 a12" order="2"/><bond id="b19" atomRefs2="a9 a13" order="1"/><bond id="b20" atomRefs2="a10 a11" order="2"/><bond id="b21" atomRefs2="a11 a19" order="1"/><bond id="b22" atomRefs2="a12 a17" order="1"/><bond id="b23" atomRefs2="a12 a20" order="1"/><bond id="b24" atomRefs2="a13 a18" order="2"/><bond id="b25" atomRefs2="a13 a21" order="1"/><bond id="b26" atomRefs2="a14 a15" order="1"/><bond id="b27" atomRefs2="a14 a25" order="1"/><bond id="b28" atomRefs2="a15 a16" order="2"/><bond id="b29" atomRefs2="a15 a23" order="1"/><bond id="b30" atomRefs2="a16 a24" order="1"/><bond id="b31" atomRefs2="a17 a22" order="2"/><bond id="b32" atomRefs2="a18 a22" order="1"/><bond id="b33" atomRefs2="a18 a26" order="1"/><bond id="b34" atomRefs2="a20 a29" order="1"/><bond id="b35" atomRefs2="a20 a30" order="1"/><bond id="b36" atomRefs2="a20 a31" order="1"/><bond id="b37" atomRefs2="a21 a32" order="1"/><bond id="b38" atomRefs2="a21 a33" order="1"/><bond id="b39" atomRefs2="a21 a34" order="1"/><bond id="b40" atomRefs2="a22 a35" order="1"/><bond id="b41" atomRefs2="a23 a36" order="1"/><bond id="b42" atomRefs2="a23 a37" order="1"/><bond id="b43" atomRefs2="a23 a38" order="1"/><bond id="b44" atomRefs2="a24 a39" order="1"/><bond id="b45" atomRefs2="a24 a40" order="1"/><bond id="b46" atomRefs2="a24 a41" order="1"/></bondArray></molecule></MChemicalStruct></MDocument></cml> | ||
<cml xmlns="http://www.chemaxon.com" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.chemaxon.com http://www.chemaxon.com/marvin/schema/mrvSchema_20_20_0.xsd"><MDocument><MChemicalStruct><molecule molID="m1"><atomArray><atom id="a1" elementType="O" x3="-4.84493" y3="1.39393" z3="-1.47056"/><atom id="a2" elementType="O" x3="4.01651" y3="4.2266" z3="-0.04928"/><atom id="a3" elementType="N" x3="0.20832" y3="1.08593" z3="0.538347"/><atom id="a4" elementType="N" x3="0.83608" y3="-1.31367" z3="-0.582773"/><atom id="a5" elementType="N" x3="0.2296" y3="3.55497" z3="1.58667"/><atom id="a6" elementType="N" x3="4.38032" y3="3.23372" z3="2.31635"/><atom id="a7" elementType="C" x3="1.20829" y3="0.06412" z3="-0.05544"/><atom id="a8" elementType="C" x3="2.64283" y3="0.707933" z3="-0.00989333"/><atom id="a9" elementType="C" x3="-1.34773" y3="0.84196" z3="0.683013"/><atom id="a10" elementType="C" x3="0.96712" y3="2.36815" z3="0.963387"/><atom id="a11" elementType="C" x3="2.47669" y3="2.16692" z3="0.63728"/><atom id="a12" elementType="C" x3="-2.32325" y3="1.24833" z3="-0.486453"/><atom id="a13" elementType="C" x3="-1.92304" y3="0.191987" z3="1.99883"/><atom id="a14" elementType="C" x3="3.84963" y3="-0.156333" z3="-0.571013"/><atom id="a15" elementType="C" x3="3.52707" y3="-1.59273" z3="-1.1312"/><atom id="a16" elementType="C" x3="2.0412" y3="-2.09599" z3="-1.10805"/><atom id="a17" elementType="C" x3="-3.87352" y3="1.00473" z3="-0.340667"/><atom id="a18" elementType="C" x3="-3.47349" y3="-0.0516133" z3="2.14461"/><atom id="a19" elementType="C" x3="3.63907" y3="3.26471" z3="0.918587"/><atom id="a20" elementType="C" x3="-1.72536" y3="1.94759" z3="-1.90699"/><atom id="a21" elementType="C" x3="-0.91168" y3="-0.257693" z3="3.27563"/><atom id="a22" elementType="C" x3="-4.44883" y3="0.354573" z3="0.974773"/><atom id="a23" elementType="C" x3="4.77867" y3="-2.54529" z3="-1.73656"/><atom id="a24" elementType="C" x3="1.67981" y3="-3.63617" z3="-1.70688"/><atom id="a25" elementType="H" x3="4.99315" y3="0.287747" z3="-0.564293"/><atom id="a26" elementType="H" x3="-3.94165" y3="-0.556733" z3="3.16157"/><atom id="a27" elementType="H" x3="0.783253" y3="4.50865" z3="1.90493"/><atom id="a28" elementType="H" x3="-0.90216" y3="3.56319" z3="1.77035"/><atom id="a29" elementType="H" x3="-1.15715" y3="3.01831" z3="-1.65835"/><atom id="a30" elementType="H" x3="-2.6096" y3="2.20108" z3="-2.73336"/><atom id="a31" elementType="H" x3="-0.929973" y3="1.18225" z3="-2.4668"/><atom id="a32" elementType="H" x3="-0.0927733" y3="-1.10889" z3="2.90827"/><atom id="a33" elementType="H" x3="-1.5512" y3="-0.753293" z3="4.21307"/><atom id="a34" elementType="H" x3="-0.311733" y3="0.724733" z3="3.72773"/><atom id="a35" elementType="H" x3="-5.65413" y3="0.158387" z3="1.10395"/><atom id="a36" elementType="H" x3="4.88189" y3="-3.59511" z3="-1.09013"/><atom id="a37" elementType="H" x3="5.87029" y3="-1.96457" z3="-1.66413"/><atom id="a38" elementType="H" x3="4.58789" y3="-2.81241" z3="-2.92955"/><atom id="a39" elementType="H" x3="2.27547" y3="-4.50865" z3="-1.06325"/><atom id="a40" elementType="H" x3="1.98072" y3="-3.72428" z3="-2.90304"/><atom id="a41" elementType="H" x3="0.4676" y3="-3.87081" z3="-1.61261"/><atom id="a42" elementType="H" x3="4.11488" y3="2.46745" z3="3.12779"/><atom id="a43" elementType="H" x3="5.21267" y3="3.98412" z3="2.55883"/><atom id="a44" elementType="H" x3="-5.87029" y3="1.14044" z3="-1.16611"/></atomArray><bondArray><bond id="b1" atomRefs2="a1 a17" order="1"/><bond id="b2" atomRefs2="a1 a44" order="1"/><bond id="b3" atomRefs2="a2 a19" order="2"/><bond id="b4" atomRefs2="a3 a7" order="1"/><bond id="b5" atomRefs2="a3 a9" order="1"/><bond id="b6" atomRefs2="a3 a10" order="1"/><bond id="b7" atomRefs2="a4 a7" order="2"/><bond id="b8" atomRefs2="a4 a16" order="1"/><bond id="b9" atomRefs2="a5 a10" order="1"/><bond id="b10" atomRefs2="a5 a27" order="1"/><bond id="b11" atomRefs2="a5 a28" order="1"/><bond id="b12" atomRefs2="a6 a19" order="1"/><bond id="b13" atomRefs2="a6 a42" order="1"/><bond id="b14" atomRefs2="a6 a43" order="1"/><bond id="b15" atomRefs2="a7 a8" order="1"/><bond id="b16" atomRefs2="a8 a11" order="1"/><bond id="b17" atomRefs2="a8 a14" order="2"/><bond id="b18" atomRefs2="a9 a12" order="2"/><bond id="b19" atomRefs2="a9 a13" order="1"><bondStereo>W</bondStereo></bond><bond id="b20" atomRefs2="a10 a11" order="2"/><bond id="b21" atomRefs2="a11 a19" order="1"/><bond id="b22" atomRefs2="a12 a17" order="1"/><bond id="b23" atomRefs2="a12 a20" order="1"/><bond id="b24" atomRefs2="a13 a18" order="2"/><bond id="b25" atomRefs2="a13 a21" order="1"/><bond id="b26" atomRefs2="a14 a15" order="1"/><bond id="b27" atomRefs2="a14 a25" order="1"/><bond id="b28" atomRefs2="a15 a16" order="2"/><bond id="b29" atomRefs2="a15 a23" order="1"/><bond id="b30" atomRefs2="a16 a24" order="1"/><bond id="b31" atomRefs2="a17 a22" order="2"/><bond id="b32" atomRefs2="a18 a22" order="1"/><bond id="b33" atomRefs2="a18 a26" order="1"/><bond id="b34" atomRefs2="a20 a29" order="1"/><bond id="b35" atomRefs2="a20 a30" order="1"/><bond id="b36" atomRefs2="a20 a31" order="1"/><bond id="b37" atomRefs2="a21 a32" order="1"/><bond id="b38" atomRefs2="a21 a33" order="1"/><bond id="b39" atomRefs2="a21 a34" order="1"/><bond id="b40" atomRefs2="a22 a35" order="1"/><bond id="b41" atomRefs2="a23 a36" order="1"/><bond id="b42" atomRefs2="a23 a37" order="1"/><bond id="b43" atomRefs2="a23 a38" order="1"/><bond id="b44" atomRefs2="a24 a39" order="1"/><bond id="b45" atomRefs2="a24 a40" order="1"/><bond id="b46" atomRefs2="a24 a41" order="1"/></bondArray></molecule></MChemicalStruct></MDocument></cml> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.