Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 20:21

    pmoura on master

    Fix LVM version requirements in… (compare)

  • 16:41

    pmoura on lgt3510stable

    (compare)

  • 13:59

    pmoura on master

    Change version to 3.51.0 stable… (compare)

  • 13:19

    pmoura on master

    Avoid some `packs` tool errors … (compare)

  • Oct 25 21:31

    pmoura on master

    Fixes and improvements for the … (compare)

  • Oct 25 19:29

    pmoura on master

    Clean up driver files for the a… (compare)

  • Oct 25 18:36

    pmoura on master

    Additional tests for the `atom_… (compare)

  • Oct 25 18:34

    pmoura on master

    Additional test for the `atom_c… (compare)

  • Oct 25 18:31

    pmoura on master

    Use portable definition of atom… (compare)

  • Oct 25 18:10

    pmoura on master

    Add tests for the `atomic_conca… (compare)

  • Oct 25 17:48

    pmoura on master

    Update `pddl` contribution plus… Update SVG diagrams (compare)

  • Oct 25 16:04

    pmoura on master

    Update the `packs` tool to use … (compare)

  • Oct 25 15:15

    pmoura on master

    Add definitions for the `atomic… (compare)

  • Oct 25 15:14

    pmoura on atomic_list_concat

    Add definitions for the `atomic… (compare)

  • Oct 25 15:13

    pmoura on atomic_list_concat

    Add `atomic_list_concat/2-3` pr… Update adapter files for the `a… Add definition for the `atomic_… and 4 more (compare)

  • Oct 25 15:06

    pmoura on atomic_list_concat

    Add definitions for the `atomic… (compare)

  • Oct 25 14:43

    pmoura on atomic_list_concat

    Add the atomic concat predicate… (compare)

  • Oct 25 14:00

    pmoura on atomic_list_concat

    Add definition for the `atomic_… (compare)

  • Oct 25 13:45

    pmoura on atomic_list_concat

    Add definition for the `atomic_… (compare)

  • Oct 25 11:41

    pmoura on atomic_list_concat

    Add definition for the `atomic_… (compare)

A Man With A Clever Disguise
@ACleverDisguise
I'm going to have to learn to power-use the debugger so that I can phase out dbg/1 messages.
Paulo Moura
@pmoura
You're missing cover(two3tree). clause in the testsobject.
(assuming that two3tree is the name of the object being tested)
A Man With A Clever Disguise
@ACleverDisguise
I had it commented out to declutter the display while there were errors showing.
Paulo Moura
@pmoura
Ah, ok.
A Man With A Clever Disguise
@ACleverDisguise
% 
% tests started at 2021-09-10, 23:42:04
% 
% running tests from object tests
% file: /home/michael/src/Playthings/Prolog/Logtalk/two3tree/tests.lgt
% 
% two3tree_new: success (in 0.0001543320000001458 seconds)
% two3tree_insert_1: success (in 0.00010364400000018037 seconds)
% two3tree_insert_2: success (in 0.00011810099999998158 seconds)
% two3tree_insert_3: success (in 0.0004362789999996508 seconds)
% two3tree_insert_4: success (in 0.00013675199999996224 seconds)
% two3tree_insert_5: success (in 0.00012492799999996862 seconds)
% two3tree_insert_6: success (in 0.00015842699999968346 seconds)
% two3tree_insert_7: success (in 0.0001948069999997415 seconds)
% 
% 8 tests: 0 skipped, 8 passed, 0 failed (0 flaky)
% completed tests from object tests
% 
% 
% clause coverage ratio and covered clauses per entity predicate
% 
% two3tree: empty/1 - 1/1 - (all)
% two3tree: insert/4 - 4/4 - (all)
% two3tree: new/1 - 1/1 - (all)
% two3tree: node4/1 - 0/1 - []
% two3tree: 6 out of 7 clauses covered, 85.714286% coverage
% 
% 1 entity declared as covered containing 7 clauses
% 1 out of 1 entity covered, 100.000000% entity coverage
% 6 out of 7 clauses covered, 85.714286% clause coverage
% 
% tests ended at 2021-09-10, 23:42:04
%
Better?
Paulo Moura
@pmoura
Good coverage. You can also easily generate a report for that. E.g. https://logtalk.org/diagrams/coverage_report.html
A Man With A Clever Disguise
@ACleverDisguise
Well, I'm very deliberately writing the tree code and the tests piecemeal, making sure that my tests are valid for each case before moving up the case ladder.
So the coverage is always going to be one or two clauses uncovered at most.
I wish I had more time to do Logtalk.
This is a whole lot more fun than C. About in the same league as Ada, but for entirely different reasons.
A Man With A Clever Disguise
@ACleverDisguise
I have one more case to contend with and then insertion is complete.
After that the low-hanging fruit of searches, iteration, etc. is in the cards. That should be trivial. (I've done most of the work needed in the tester helpers for validating the tree structure.)
Once that's done, the Big One: deletion.
That's a real pig of an operation in 2-3 trees.
A lot of libraries that do 2-3 trees don't do deletion, but I suspect it will be easier in a declarative language than an imperative.
Paulo Moura
@pmoura
It should be possible to evolve your implementation so that it would eventaully comply with the dictionaries library protocol.
A Man With A Clever Disguise
@ACleverDisguise
That is the plan.
:- object(two3tree, implements(dictionaryp),
                    extends(term)).
…
Paulo Moura
@pmoura
👍
Paulo Moura
@pmoura
Got a short break and tried to reproduce the bug with the dbg/1 calls. No luck so far. It works fine in my experiments.
With the debug flag turned off, I call tests::run. No debug messages. I turn of the flag, run the tests again and get debug messages. Tried with dbg/1 calls from cleanup/0, tests, and test auxiliary predicates.
Paulo Moura
@pmoura
No chance that you're typing set_prolog_flag instead of set_logtalk_flag?
A Man With A Clever Disguise
@ACleverDisguise
Nope. Definitely typing set_logtalk_flag. And using current_logtalk_flag to get the setting.
Paulo Moura
@pmoura
You will need to provide all the necessary code and steps to reproduce the issue then.
A Man With A Clever Disguise
@ACleverDisguise
Will do. Probably on the weekend, though. I don't have a huge amount of time in the week to do things anymore. :-/
Paulo Moura
@pmoura
Resumed work on the Logtalk package manager. Details at https://github.com/LogtalkDotOrg/logtalk3/discussions/118
Paulo Moura
@pmoura
Logtalk 3.51.0 released