coordgenlibs icon indicating copy to clipboard operation
coordgenlibs copied to clipboard

Some coordgen templates don't match themselves

Open d-b-w opened this issue 5 years ago • 2 comments

There are some coordgen templates that don't match themselves! If I import the coordgen templates.mae file into Maestro and display the structures in the 2D viewer, the 2D view clearly doesn't match the input template.

This affects templates (0 indexed):

 [1, 8, 19, 20, 22, 43, 53, 65, 66, 67]

Here's an example. White is the coordinates generated by coordgen, black is the template structure:

coordgen_bad_template

Furthermore, some of the templates match other templates instead of themselves! This means that that some templates will never be used, and so only part of the template is actually honored. This affects templates (0 indexed):

[18, 27]

d-b-w avatar Feb 14 '20 06:02 d-b-w

this happens because we now try to first open one ring and generate the rest of the coordinates and when we can't do that resort to templates. I'm actually happy with the result in this case but I do think that it'd be worth to search for templates as a first step instead and have a look at how that impacts performance

ZontaNicola avatar Feb 14 '20 08:02 ZontaNicola

If it doesn't have a big performance impact, it seems like the "expected" behavior would indeed be to check the templates first.

Are templates allowed to match parts of a set of fused rings? I could imagine that leading to awkward depictions if we apply the templates first.

greglandrum avatar Feb 14 '20 09:02 greglandrum