Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
Reaction mapping memory leak
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  4 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Daniel Lowe  
View profile  
 More options Jun 28 2011, 11:31 am
From: Daniel Lowe <daniel_lowe...@yahoo.co.uk>
Date: Tue, 28 Jun 2011 08:31:21 -0700 (PDT)
Local: Tues, Jun 28 2011 11:31 am
Subject: Reaction mapping memory leak
IndigoObject reaction = indigo.createReaction();
reaction.addProduct(indigo.loadMolecule("[Au]"));
reaction.addReactant(indigo.loadMolecule("OC(=O)C=C"));
reaction.addReactant(indigo.loadMolecule("OC(=O)CC(=C)C(O)=O"));
reaction.addReactant(indigo.loadMolecule("C(C=C)N"));
reaction.addReactant(indigo.loadMolecule("C(=C)CCC(=O)O"));
reaction.addReactant(indigo.loadMolecule("C(=C)S(=O)(=O)O"));
reaction.addReactant(indigo.loadMolecule("C(=CC1=CC=CC=C1)S(=O)
(=O)O"));
reaction.addReactant(indigo.loadMolecule("S(=O)(=O)(O)C=C"));
reaction.addReactant(indigo.loadMolecule("[O-]C(=O)C=C"));
reaction.addReactant(indigo.loadMolecule("S(=O)(=O)(O)CC=C"));
reaction.addReactant(indigo.loadMolecule("[O-]C(=O)C=C"));
reaction.addReactant(indigo.loadMolecule("[O-]C(=O)C=C"));
reaction.automap("discard");

causes all the system's memory to be used up and after a while
"array: reserve(): no memory" to be displayed.

Obviously this is not an actual reaction; I am actually intending to
use the result of the mapping as a heuristic which would indicate that
this is not a valid reaction.

I think the memory leak is connected to the number of reactants.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Savelyev Alexander  
View profile  
 More options Jun 29 2011, 4:33 am
From: Savelyev Alexander <asavel...@ggasoftware.com>
Date: Wed, 29 Jun 2011 01:33:32 -0700 (PDT)
Local: Wed, Jun 29 2011 4:33 am
Subject: Re: Reaction mapping memory leak
Hi Daniel
Thank you for the bug report. The bug was fixed. The new version will
be available in the upcoming indigo release.
It was not actually a memory leak. The AAM algorithm was trying to
calculate all reactants permutations thus the "no memory" exception
was raised.

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Daniel Lowe  
View profile  
 More options Jun 29 2011, 6:18 am
From: Daniel Lowe <daniel_lowe...@yahoo.co.uk>
Date: Wed, 29 Jun 2011 03:18:18 -0700 (PDT)
Local: Wed, Jun 29 2011 6:18 am
Subject: Re: Reaction mapping memory leak
Is this a change in behaviour? Previously the algorithm seemed to be
matching reactants sequentially which in the case of a reaction with
only 1 product would make the problem only order N.
If every permutation does need to be considered i.e. the problem is N!
then throwing an exception if there are too many reactants/products
and AAM is performed could be an answer although this form of AAM
should probably not be the default.

On Jun 29, 9:33 am, Savelyev Alexander <asavel...@ggasoftware.com>
wrote:


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Savelyev Alexander  
View profile  
 More options Jun 29 2011, 7:08 am
From: Savelyev Alexander <asavel...@ggasoftware.com>
Date: Wed, 29 Jun 2011 04:08:18 -0700 (PDT)
Local: Wed, Jun 29 2011 7:08 am
Subject: Re: Reaction mapping memory leak
This is not a change in behaviour. The upper bound for permutations
was added only. The algorithm uses permutations to calculate all
possible maps and chooses the best one. So, the fix may affect only
difficult reactions with a large reactants number. This is not a
reason to throw an exception in our opinion. But such a solution with
raising errors may be added in the future (for example, optional)

 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »