libera/#clasp - IRC Chatlog
Search
16:30:30
Bike
not sure if i did something wrong, but source locations don't seem to be working right? M-. gives me that characterp error. the source filename for core:file-column is given as #p"sys:src;core;lispStream.cc", which seems like it ought to be uppercase
16:32:07
Bike
TAGS also being weird... i took the etags thing out of my config a while back, but it's still failing to find pretty uncomplicated definitions
16:36:56
yitzi
I didn't change anything for source locations there. That is a different branch. The scraper source reference like #P"sys:src;core;lispStream.cc" won't work because of the mangled case.
16:43:48
yitzi
The logical pathname stuff for source locations I have mentioned is in `yitzchak_logical-kernel-paths`. In main, the source references that come from the scraper are probably broken logical paths. The rest are stils absolute paths.
16:45:32
yitzi
Yes, well....that would be the easy way to say it. I was going for convoluted and obscure.
16:46:37
yitzi
I still need to test the logical source references for an installed clasp and for snapshots.
17:05:33
Bike
https://github.com/clasp-developers/clasp/issues/1348 minor: not sure if this is a bug
17:19:32
yitzi
I wouldn't be surprised that clisp's printer (or pretty printer) behaves differently from other implementations. As I recall even the pretty printer is not derived from the original XP code and has a lot implementation in C code.
17:21:40
Bike
it's not actually a pretty printer thing. i replicated this behavior in clisp by giving it a longer repl prompt.
18:53:24
kpoeck
the asdf self test complain - correctly that we have code-weight.asd in profiler, once https://github.com/clasp-developers/clasp/blob/main/src/profiler/code-weight.asd and once in https://github.com/clasp-developers/clasp/blob/main/src/profiler/code-weight/code-weight.asd. Surprisingly the file code-weight.lisp is also different in the two places.
18:53:24
kpoeck
And file there is package definition n one asd that doesn't seem to be used. The description of this system is: Compile-file files with :output-info t and generate .ll files and then pass a list of .ll files to the analyze-code function.
18:54:29
kpoeck
Can't see that analyze-code s called anywhere so probably just an utility. Any preference on how to clean that up? If not I would just do a PR with my best educated guess
19:15:03
kpoeck
yes, and probably code-weight.lisp for consistency. And if we create a package #:code-weight, we might as well use it in code-weight.lisp
22:14:58
Colleen
slyrus__: drmeister said 16 hours, 44 minutes ago: Could you ping me when you are online? I downloaded your smiles parser and started modifying it to work with our molecule/atom graphs. I kept your copyright notification of course.
22:15:43
drmeister
Quickly: are you ok if I borrow and modify your smiles parser to work with our molecule graphs?
22:43:45
yitzi
Looks like I've got the logical source references to work in all the cases I can think of...dev tree, installed clasp, and snapshots. Now let the GitHub workflow run the tests.
22:55:54
drmeister
yitzi: What do you think is the timeline for cando to automatically update itself?
22:57:24
drmeister
yitzi: Ok - that sounds great. Also - does Bike know about your solution for logical pathnames and mixed case and underscores using pathname translations?
22:59:28
drmeister
I may have been the last person to hear about it - if not - give a quick description because it sounded generally useful.
23:00:20
yitzi
Yes, he knows the jist of it. I didn't describe all the final details to him. I'll make sure to describe exactly what is going on at a minimum in the PR.
23:03:51
yitzi
I just tested it with eclasp. Works no problems. Piles in the the logical translations for all the cando source code as needed.
23:06:13
yitzi
I also added the code that creates the translations to the common system "ninja" which the scraper also uses. The system name is probably not appropriate anymore. It has the code for writing ninja build files and other things useful for making them.
23:28:01
drmeister
https://tenor.com/view/the-universe-tim-and-eric-mind-blown-mind-blown-meme-mind-explosion-mind-explosion-meme-gif-18002878
23:28:48
yitzi
Haha! The homebrew pr hasn't been merged yet. https://github.com/Homebrew/homebrew-core/pull/97618