Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Segmentation fault: Open *.smi + activate debug #45

Closed
ghutchis opened this issue Sep 5, 2007 · 4 comments
Closed

Segmentation fault: Open *.smi + activate debug #45

ghutchis opened this issue Sep 5, 2007 · 4 comments

Comments

@ghutchis
Copy link
Owner

ghutchis commented Sep 5, 2007

  • Opening *.smi files produces a garbled "View", activating engine "Debug" additionaly segfaults avogadro.

Reported by: soc88

@ghutchis
Copy link
Owner Author

Logged In: YES
user_id=1500648
Originator: NO

can you attach some *.smi files to this bug so I can have something to test. I don't even know what an smi file looks like.

Original comment by: @milkypostman

@ghutchis
Copy link
Owner Author

Logged In: YES
user_id=684210
Originator: NO

http://filext.com/file-extension/SMI tells me it is a SMILES format. Googling even more gives me these files:

http://www.perlmol.org/examples/combinatorial_enumeration/amines.smi

http://www.perlmol.org/examples/combinatorial_enumeration/acid_chlorides.smi

Original comment by: @cniehaus

@ghutchis
Copy link
Owner Author

Logged In: YES
user_id=684210
Originator: NO

Ok, I played around with Avo. I loaded both mentioned files and cannot makes Avo crash.

But yes, the View is pointless: All I get is ONE atom. I guess it should be four molecules at a time for amines.smi...

kde4@moinmoin:~/build/avo> cat amines.smi
CN methylamine
CCN ethylamine
CCCNC propylamine
c1ccccc1N aniline

Original comment by: @cniehaus

@ghutchis
Copy link
Owner Author

Logged In: YES
user_id=1883751
Originator: YES

Weird, I tested 15 different *.smi from the chemical-structures-2.0.1 package. Result:

  • No one produced a garbled output.
  • Debug didn't crash it.
    It seems this bug has disappeared in more recent releases.

I think it's the best to close the bug with resolution "Out of Date", if the bug appears again, we can reopen it.

Original comment by: soc88

@ghutchis ghutchis self-assigned this Apr 24, 2016
ghutchis added a commit that referenced this issue May 25, 2016
Update website URLs for new schemes (to easily allow redirects).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant