Skip to content
This repository has been archived by the owner on Sep 1, 2021. It is now read-only.

Crystal structure builder duplicates atoms #507

Open
cryos opened this issue Mar 5, 2010 · 4 comments
Open

Crystal structure builder duplicates atoms #507

cryos opened this issue Mar 5, 2010 · 4 comments
Assignees

Comments

@cryos
Copy link
Owner

cryos commented Mar 5, 2010

Greetings. Your program is really fantastic thanks for all your hard work.

Here is the bug:
When I enlarge the C axis using the super cell builder on molecules such as the one attached, the resulting structure has duplicate atoms. You can check by using the attached file (I got atom positions from paper: Imberty 1988 Biopolymers v27 p1206) and build up the c axis in super cell builder (aka make c=1 and generate) and you can see if you select rings or if you look in cartesian editor that there are some atom duplicates. This occurs because there are slightly more atoms in this basic structure than should be in one unit cell, so those get duplicated. I have found no solution to this other than random cutting, or using excel to easily find duplicates and erase them. It would be great if you put in a routine to get rid of duplicate atoms, either in the cell builder or seperately.

Thanks so much.

Reported by: *anonymous

@cryos
Copy link
Owner Author

cryos commented Mar 5, 2010

Starch Helix type B transcribed from Imberty 1988

Original comment by: nobody

@cryos
Copy link
Owner Author

cryos commented Mar 10, 2010

I can confirm this and it looks like the supercell extension should be removing duplicates already? Marcus, any ideas here?

Original comment by: @dlonie

@cryos
Copy link
Owner Author

cryos commented Aug 23, 2011

Sorry for the delay. I'm taking over the supercell generation feature, so I'll check this bug before the 1.1 release.

Original comment by: @ghutchis

@MSoegtropIMC
Copy link

I just wanted to note that this bug is still there in Avogradro 1.2.0. It can be reproduced as follows:

  • File/Import/Crystal/C-graphite
  • Build/Super Cell Builer ...
  • Enter multiplicity 10x10
  • export file as .CML

The resulting file should have 400 atoms, but it has 500 atoms. If I copy paste the individual layers, I find that the lower layer has 200 atoms, the upper layer has 300 atoms.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants