freenode/#clasp - IRC Chatlog
Search
14:47:01
drmeister
You mean three different ways that source info is being tracked? Only one matters now - CST. The rest are to be cleaned out.
16:11:45
attila_lendvai
hi! drmeister, Bike, what's the deal with the dev-cst merge? I have that merge from several weeks ago... I guess it would need to be updated by now, possibly heavily, especially with some of the git history weirdness that sometimes happen
16:45:07
Bike
should i just, download alexandria or something, or should we mark that as an actual dependency
16:49:49
attila_lendvai
Bike: yep. I worked quite a lot on merging dev and cst and resolving the issues. it would be a pity if it got wasted and one of you ends up redoing it (as opposed to just review it). it's still from those times when I finished refactoring the wscript. (e.g. I fixed this missing alexandria issue in the merge, mps was twice, which is probably a merge issue itself)
16:50:40
attila_lendvai
FTR, this is the commit: https://github.com/attila-lendvai/clasp/commit/8bce100acc71972b581e728e14df08ab6f29b569 (but github.com shows way much larger diff, the gitk view is much more reasonable)
16:52:01
attila_lendvai
Bike: by now it's probably out of date, so I guess if you're ready to look into it, then maybe first I should try to rebase the merge, and if you find it ok after a review, you can just push it as official cst
16:52:48
attila_lendvai
yeah, those deletions are the cleanup in the dev branch. github shows it as part of the merge, which is... useless.
16:53:43
attila_lendvai
I have noticed at least one bug in the C++ code while I was resolving the conflicts, so it gives some hope that I was not completely clueless, but... you can bet I'm more unfamiliar with the entire codebase than you... :)
16:56:53
attila_lendvai
there were not that many conflicts, but still numerous and complex enough, and the build was still slow enough to turn the merge into a longer process. I learned a lot about the codebase in the process, though.
16:59:38
attila_lendvai
ok, I'll be back next week with more time, and update the merge commit to merge the latest HEADs. or maybe record another, new merge? that may be a better idea, but I need to think about that first (it affects what the merge diff looks like, i.e. how readable it is)
17:02:10
Bike
it's big enough that it's going to be difficult either way. maybe you could mark things you're not sure about? stuff like deleting whole files should be no problem, etc
17:02:35
attila_lendvai
Bike: in gitk you only see the relevant part of the merge, the context around actual conflicts
17:06:47
attila_lendvai
Bike: just fix and push it to cst. that won't be a major showstopper in the merge...
17:06:52
drmeister
Yeah - the cst merge is taking longer than I thought it would. Inlining was problematic and benefits from some cleavir changes.
17:07:33
attila_lendvai
drmeister: been busy with... life. learning about nutrition and getting my mom off of insulin, stuff like that. I'll have some hacking time next week.
17:10:04
attila_lendvai
Bike: just copy the relevant part of update_dependencies from dev to cst in the wscript
17:10:19
drmeister
Ok - I cloned alexandria and I'll add it to the wscript file to pull this hash: e5c54bc30b0887c237bde2827036d17315f88737
17:10:53
drmeister
Asking yes on the cloning and adding to wscript file - you don't need to check the hash - it's the latest.
17:12:19
attila_lendvai
drmeister: it's actually very good news. apparently type 2 diabetes is reversible, and surprisingly easily
17:23:58
drmeister
I found the problem with MPS. ClassRep_O inherits from Instance_O - for some reason ClassRep_O instances are being created with an Instance_O stamp in the header.
17:27:50
attila_lendvai
drmeister: exercise is actually not crucial in the reversal, it's just healthy otherwise. the key step is fasting, so that the body eats up the accumulated fat that is clogging the internal organs, especially the liver and the pancreas. for some lucky (and determined) people 3 weeks of waterfast gets them off of a decade of insulin use. for others it can be a longer process.
17:29:50
attila_lendvai
drmeister: no worries, it's far from a widely accepted view among doctors, but it's proven by many publications in the last few years
17:33:22
Bike
https://irclog.tymoon.eu/freenode/%23clasp?around=1513736075#1513736075 something something
17:35:26
attila_lendvai
my mantra: kludges are perfectly fine -- *if* they are marked as kludges, and *if* they are leading to explicit and early failures only. we even have red coloring in our emacs for KLUDGE, TODO, FIXME
17:35:57
attila_lendvai
and the reason is primarily psychological, along the lines of the broken window idea
17:36:08
drmeister
Classes are instances of Instance_O. ClassRep_O are for classes that are exposed with clbind - it is derived from Instance_O because it needs extra information about the C++ class that is being exposed.
17:50:01
drmeister
I figured out last night that it will be trivial to add custom pools for builtin C++ objects of specific classes.
17:50:37
drmeister
It's a little bit of template specialization code and an API to add and access custom allocators to the thread-local storage.
18:21:07
drmeister
I have to run the static analyzer again. Uncommenting that line creates some global variables that the GC needs to be aware of.
18:44:16
drmeister
It's disturbing - there is something wrong with the debug info that we are generating - I believe
19:33:00
shiho
drmeister: I tried (load-off "atomic_ions.lib") and got the error. "Could not determine element for atomic number 94"