Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use teahouse #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Use teahouse #1

wants to merge 4 commits into from

Conversation

rcoder
Copy link

@rcoder rcoder commented Aug 20, 2022

This mainlines the change from nostr-tools to teahouse, and updates the commit used of the latter to fix a few early compatibility bugs.

rcoder and others added 4 commits August 19, 2022 16:34
also:
  remove logging
  fix typo
  move debug info to top of page for now
increase bulkPut batch size to speed up sync
@Giszmo
Copy link
Owner

Giszmo commented Sep 21, 2022

Back when I last checked this PR it was promising for performance but broke several features. Now master moved on and the PR has several merge conflicts with severe issues around the introduction of a SharedWorker I suspect. As the introduction of the SharedWorker did fix some performance issues but teahouse still was many times faster, I would love to see this adapted to the changed master with bugs fixed but I'm not sure if the library is feature complete now.

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.

3 participants