The Wayback Machine - https://web.archive.org/web/20201107064347/https://github.com/ipfs-shipyard/ipfs-desktop/issues/1647
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

Menu items for enabling pubsub and automatic GC #1647

Open
lidel opened this issue Sep 16, 2020 · 3 comments
Open

Menu items for enabling pubsub and automatic GC #1647

lidel opened this issue Sep 16, 2020 · 3 comments

Comments

@lidel
Copy link
Member

@lidel lidel commented Sep 16, 2020

I was looking into starting go-ipfs with additional flags: --enable-pubsub-experiment (pubsub) and --enable-namesys-pubsub (IPNS over pubsub) and the main concern I've found was use of resources:

In order to enable pubsub by default in go-ipfs, we need to find some way for pubsub to not take up a bunch of resources when not in-use.
libp2p/go-libp2p-pubsub#332

So if we want pubsub and fast IPNS enabled, we need to pay the cost of higher resource utilization.
We could measure it and decide if its worth the added value, or prioritize a fix.

Updated: or at the very least, we could give user UI for enabling that with one click

Updated again: See mockup for placement of menu items for pubsub and automatic GC (note this also includes not-yet-implemented "Ask when opening" menu item, likely to land first in #1646)
image

@aschmahmann are there any other concerns that I don't see?

@autonome FYSA this also impacts go-ipfs in Brave, especially the speed of ipns://

@shaikh-shahid
Copy link

@shaikh-shahid shaikh-shahid commented Sep 18, 2020

I think if we have a UI based switch to turn on/off the flags such as Pubsub and GC etc then that would be better for end users as well. I am currently working on a framework that does require IPFS in the backend and needs the PUBSUB to be enabled. Since the framework also consists of demos for general public as well, it's difficult for users to find the configuration and update the flags.

@lidel
Copy link
Member Author

@lidel lidel commented Sep 21, 2020

@shaikh-shahid that is very useful feedback, thank you.

I think we could avoid enabling pubsub by default by adding on/off checkbox settings for pubsub (and automatic GC) to the Desktop app menu. Toggling would automatically restart the deamon with the new setting.

@lidel lidel changed the title Enable pubsub by default Menu item for enabling pubsub Sep 21, 2020
@jessicaschilling jessicaschilling changed the title Menu item for enabling pubsub Menu items for enabling pubsub and automatic GC Sep 21, 2020
@jessicaschilling
Copy link
Contributor

@jessicaschilling jessicaschilling commented Sep 21, 2020

Note: Added mockup of menu item placement in original comment.

Unless @aschmahmann or @autonome have objections, I believe this is ready to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.