spritely/goblins
4
58
Fork
You've already forked goblins
4

WIP: Draft: Replace methods with a more performant version #594

Draft
Marie-Joseph wants to merge 1 commit from update-methods into main
pull from: update-methods
merge into: spritely:main
spritely:main
spritely:hashtable-actor
spritely:vat-log-resize-metacommand
spritely:fix-ocapn-ci
spritely:fix-796
spritely:support-zero-values
spritely:maybe-fix-ocapn-ci
spritely:document-wants-partial
spritely:prelay-docs
spritely:aurie-docs
spritely:lazy-captp
spritely:fast-sealers-fast-spawn
spritely:more-handoff-recieve-checks
spritely:ocapn-test-suite
spritely:add-keys-values-to-ghash-actor
spritely:docs-captp-ref-passing
spritely:fix-nested-with-vat
spritely:fix-281
spritely:captp-crossed-hellos
spritely:fix-243
spritely:sleepymaps
spritely:hootify-tests
spritely:move-define-actor-docs
spritely:websocket-netlayer
spritely:hoot-deprecation-warning
spritely:add-new-on-macros
spritely:replace-while-with-do
spritely:unenclose-syscaller
spritely:new-devel
spritely:aurie-netlayer
spritely:devel
spritely:document-value-extraction
spritely:aurie-churn-fix
spritely:io-actor
spritely:cwebber-aurie-foundations
spritely:ci-code-coverage
spritely:0.12-news
spritely:gnutls-package
spritely:oops-all-goblins
spritely:relay-maybe-simplified
spritely:fake-indentation-fix
spritely:captp-remove-wants-partial-from-op-listen
spritely:mvre-wtf
spritely:multi-value-return-everywhere
spritely:simple-pick
spritely:remove-ports-dynamic-wrap
spritely:fast-spawn
spritely:additional-vat-events
spritely:onion-netlayer-temp
spritely:klugey-queue-captp-message
spritely:rest-of-captp
spritely:facet
spritely:ghash-pretty-print
spritely:fixing-syrup
spritely:fake-netlayer
spritely:onion-netlayer
spritely:string-formatting-errors
spritely:remove-waiter-cruft
spritely:lmethods
spritely:current-scheduler-kludge
spritely:hygienic-define-recordable
spritely:convert-captp
Marie-Joseph commented 2024年10月28日 15:10:24 +01:00 (Migrated from gitlab.com)
Copy link

This version of methods does not support extend-methods. CI will fail for this reason. I don't know enough macro witchery to make this version of methods work with extend-methods.

(reproducing the message from the original, accidentally-closed MR)

No rush to merge this, but I wanted to go ahead and open this PR before I forget about it for another year... Especially since we'd like the most performant version of Goblins possible for the Shepherd ;)

While this update started with work I did about a year ago, David Thompson has helped significantly over the course of the last couple days and really ought to get the majority of the credit for the actual performance parts. I mostly just glued stuff together.

This version of `methods` does not support `extend-methods`. CI will fail for this reason. I don't know enough macro witchery to make this version of `methods` work with `extend-methods`. (reproducing the message from the original, accidentally-closed MR) No rush to merge this, but I wanted to go ahead and open this PR before I forget about it for another year... Especially since we'd like the most performant version of Goblins possible for the Shepherd ;) While this update started with work I did about a year ago, David Thompson has helped significantly over the course of the last couple days and really ought to get the majority of the credit for the actual performance parts. I mostly just glued stuff together.
This pull request has changes conflicting with the target branch.
  • goblins/actor-lib/methods.scm
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin update-methods:update-methods
git switch update-methods
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
spritely/goblins!594
Reference in a new issue
spritely/goblins
No description provided.
Delete branch "update-methods"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?