-
Notifications
You must be signed in to change notification settings - Fork 461
Update academic-papers.md #1242
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
Conversation
willscott
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. This order looks good. Maybe worth noting that this one has been peer-reviewed / published at SIGCOMM '22?
|
I think the current order makes sense, having the original IPFS paper at the top. Since the papers are not ordered by date, I would add in brackets the publication year. |
johnnymatthews
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
|
TMoMoreau
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
|
Yup, looks good to me as well! Thanks @lidel for catching this. A couple of thoughts (non-blocking for this PR):
|
|
@yiannisbot Good suggestions, could you open up a separate issue for it? Just in an attempt to keep this PR as clean and simple as possible :) |
|
Let's continue in #1247 |
@yiannisbot @willscott ok to add the more recent paper to https://docs.ipfs.tech/concepts/further-reading/academic-papers/?
Any preference with regard to ordering?
Anything else that should be added?