freenode/#clasp - IRC Chatlog
Search
12:28:58
kpoeck
Instructions are in https://github.com/clasp-developers/clasp/wiki/Build-Instructions
14:40:39
drmeister
libelf is a new dependency on linux - I think I forgot to add it to the documentation - adding it now.
15:34:55
drmeister
shiho: The TagSet_O object will accept a logical - the problem may be in the builder protocol - I don't think it's constructing a proper TagSet_O object when given a smarts string like: "[something;something]1"
15:43:03
drmeister
That's where you set the atom-test to nil - what other values are available in the args?
15:45:48
drmeister
What about here: https://github.com/drmeister/cando/blob/dev/src/lisp/smarts/smarts.lisp#L215
15:46:10
drmeister
That's where we relate tag-set nodes to other things. Does this only get invoked when the other thing is an atom-test?
15:48:45
drmeister
I'm trying to figure out if the parser is not working properly here or we are just not handling what the parser is giving us.
15:49:40
drmeister
Well, here you go - this method is probably what is being called - and it doesn't create anything.
15:51:34
drmeister
These tests that you are running - can you set them all up in a source file so that we can run them all?
15:51:56
drmeister
You are doing good testing work here - we should accumulate the tests for the future.
15:52:54
drmeister
Meanwhile - usha wants to start testing the AM1-BCC charge calculations on compound libraries - comparing to Antechamber
15:53:51
shiho
The test comes from am1-bcc.lisp. If the test doesn't work, it means am1-bcc doesn't work...
15:55:03
drmeister
But still - it would be good to duplicate these tests into one file that tests SMARTS
15:59:37
drmeister
And then scymtym joins. Hi scymtym - we are just talking about the SMARTS parser and the builder protocol. It's more cando integration than anything else so we were going to take it into a private channel. We can keep it here if you are interested.
16:01:49
drmeister
Ok - we are just sorting out how to deal with things like "[C]1" vs "C1" - they should be handled the same way but aren't at the moment.
16:03:19
drmeister
I also overloaded them to put tags on atoms rather than using them to test for ring closures.
16:05:12
scymtym
i suggest writing down what the different syntactic forms /should/ mean, then writing corresponding unit tests and finally changing the parser
16:05:54
scymtym
i did the unit-test-before-code-change approach for the recent changes and fixes you requested
16:06:07
drmeister
We are discovering shortcomings in the old parser - it's really good that we are switching to this new one.
16:07:48
scymtym
ok, i hope it's up to the task. these highly ambiguous grammars with operator precedence are easy to get wrong
16:17:27
scymtym
regarding the issue at hand: (esrap:trace-rule 'language.smarts.parser::smarts :recursive t) (language.smarts.parser:parse "C1" 'list) is often enough to see what's going on
19:09:08
Bike
serial build is messed up for me. i know that't not a big priority or anything, but still. i'm guessing it's something to do with inline.lisp being in front.
19:10:38
drmeister
Does that make sense? inline.lisp is already loaded and we are compile-filing it. Does compile-file'ing inlined functions change something in the system?
19:13:12
Bike
the clasp executable builds, but if you run it it complains about there being no class called NAMED-FUNCTION-AST
19:13:25
Bike
which would make sense if it tries to load the inline.fasl before the class is defined.
19:14:49
drmeister
Oh - so at startup the inline toplevel forms are running before everything else. They are running in order of how they are being compile-file'd.
19:55:36
shiho
drmeister: Now, the test stopped in the line "this->_TagLookup->setf_gethash(tag, a);" in ChemInfoMatch_O::defineAtomTag.
19:56:16
shiho
I remember we changed core::HashTableEqual_sp _TagLookup to core::HashTableEql_sp _TagLookup