Thank you for your attention!

Seems like the bug is still not fixed by your patch. Trying
/obabel -isdf test.sdf -osdf --filter "MyProp<-1 & MyProp<-1.5"/
with the same SDF file still gives "0 molecules converted".

In this example, while parsing the first statement of the filter
(/MyProp<-1/), after line 267 in descriptor.cpp (/optionText >> val;/), the
filter stream (/optionText/) is
/&MyProp<-1.5/
so it neither has EOF bit set, nor starts with ')'. This leads (again) to
setting /val/ to NAN, which in turn causes identification of the value as
string.

What is actually the purpose of these lines?
/if(!optionText.eof() && optionText.peek()!=')')
  val = std::numeric_limits<double>::quiet_NaN();/

Oleksii



--
View this message in context: 
http://forums.openbabel.org/Molecular-descriptor-filter-does-not-work-properly-tp4656057p4656064.html
Sent from the openbabel-devel mailing list archive at Nabble.com.

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
OpenBabel-Devel mailing list
OpenBabel-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbabel-devel

Reply via email to