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

Add system requirements to README #5137

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

Mr0grog
Copy link
Contributor

@Mr0grog Mr0grog commented Jun 19, 2018

This adds a short paragraph about system requirements to the “install” section of the README (that seemed like the most applicable place). While writing this, I wondered if it might make sense to add a paragraph about disk space:

You should also make sure to have about 1.3x the disk space as the amount of data you’d like to be able to store — IPFS takes up a little bit of extra space keeping metadata about your files and about peers on the network. You can also choose different ways to store your data, which will impact the amount of storage space required.

…but I wasn’t sure if that was actually necessary to say or even very accurate (There’s definitely a lot of fudge factor here, and I’m not even sure the 1.3x number is even a great reasonable approximation), so I left it out. I can add it in if people think it’s useful, though.

Fixes #5136.

License: MIT
Signed-off-by: Rob Brackett <[email protected]>
@Mr0grog Mr0grog requested a review from Kubuxu as a code owner June 19, 2018 00:00
@Stebalien
Copy link
Member

…but I wasn’t sure if that was actually necessary to say or even very accurate (There’s definitely a lot of fudge factor here, and I’m not even sure the 1.3x number is even a great reasonable approximation), so I left it out. I can add it in if people think it’s useful, though.

So, IPFS actually tends to use less space, but that really depends on what you're storing. I'd just say that IPFS may use more or less space, depending on what you're storing.

@Mr0grog
Copy link
Contributor Author

Mr0grog commented Jun 19, 2018

Sounds like I should just leave it out (i.e. leave the PR as-is) then.

@whyrusleeping whyrusleeping merged commit be20903 into ipfs:master Jun 22, 2018
@Mr0grog Mr0grog deleted the docs/5136-resource-requirements branch June 22, 2018 19:25
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