Skip to content

lib_pln: port queue/dedup improvements from benchmark variant#127

Closed
rTreutlein wants to merge 2 commits intofeat/lib-remaining-changes-20260220-085516from
lib-pln-queue-dedup-improvements
Closed

lib_pln: port queue/dedup improvements from benchmark variant#127
rTreutlein wants to merge 2 commits intofeat/lib-remaining-changes-20260220-085516from
lib-pln-queue-dedup-improvements

Conversation

@rTreutlein
Copy link
Collaborator

Summary

  • port heap-based queue limiting and best-confidence selection into lib/lib_pln.metta
  • add incremental dedup helpers and use them during derive expansion to reduce repeated candidates
  • replace stamp disjoint cross-product scan with recursive membership checks
  • remove debug-style derive noise from the benchmark variant while preserving core behavior

Validation

  • ran sh run.sh ./examples/pln_roman.metta as a smoke check

Copy link
Collaborator

@patham9 patham9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thank you very much, this looks great actually!
Any chance we can transfer this PR to https://github.com/trueagi-io/PLN?
A few days ago wet moved lib_pln back there, we didn't yet remove it from PeTTa (removing things not until v1.1 version)

@rTreutlein rTreutlein force-pushed the lib-pln-queue-dedup-improvements branch from 3d35b16 to 6c24c32 Compare February 20, 2026 08:10
@rTreutlein rTreutlein changed the base branch from main to feat/lib-remaining-changes-20260220-085516 February 20, 2026 08:10
…260220-085516

Update lib imports and roman tuple helpers
@patham9
Copy link
Collaborator

patham9 commented Feb 20, 2026

I just tried to override lib_pln of https://github.com/trueagi-io/PLN with the lib_pln of this PR. The more extended test cases there do not pass anymore, but maybe it is minor issues.

@rTreutlein
Copy link
Collaborator Author

I'll have a look at that.

@rTreutlein rTreutlein force-pushed the lib-pln-queue-dedup-improvements branch from 6c24c32 to 80b842f Compare February 20, 2026 13:08
@rTreutlein
Copy link
Collaborator Author

Closing as superseded: the equivalent lib_pln changes have been merged in trueagi-io/PLN.

@rTreutlein rTreutlein closed this Feb 20, 2026
@rTreutlein rTreutlein deleted the lib-pln-queue-dedup-improvements branch February 20, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants