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

Whitepaper economics update and edits #237

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

Conversation

nottoseethesun
Copy link
Contributor

@nottoseethesun nottoseethesun commented Feb 17, 2020

Overview

I took care of the remaining to-do items; pending PR approval, this can now be a release candidate.

  • Extracted the chart image from the earlier pdf, added the chart image as a file, and included it in the markdown document in the expected place.
  • Fixed the Slack link.
  • Cleaned up a couple grammatical issues, and formatted a single whitespace.

Added Automatic Creation of PDF from Markdown Source (Feb. 22, 2020)

Provides automatic pdf generation, as a GitHub Build Artifact (in a zip file), from the markdown source code, whenever that markdown is changed.

Here's what the result looks like: https://github.com/nottoseethesun/Gridcoin-Tasks/actions/runs/43624529

Currently, the zip file is the only option for the official GitHub Action used.

Later, we could consider using something like this: https://github.com/marketplace/actions/gh-pages-deploy to publish automatically to gh-pages.

@barton2526
Copy link
Member

Please add the .DS_Store file to your git ignore list. This is a mac autogenerated attribute file.

@nottoseethesun
Copy link
Contributor Author

@barton2526 Thanks, done; yes I needed to configure that on my machine.

@wilkart
Copy link

wilkart commented Feb 18, 2023

Is it going to be merged?

@nottoseethesun
Copy link
Contributor Author

nottoseethesun commented Feb 18, 2023

Is it going to be merged?

An earlier PR was merged: #236

As for this one, I think so, one of these years. There was a delay of a couple years while the Gridcoin website was being re-done. I raised the topic over half a year ago in the Discord and some members were responsive, but still no action.

None of this content is published yet at the links on the Gridcoin website. It's still the old whitepaper from many years ago.

@jamescowens
Copy link
Member

Can we figure out where we are with this and get the whitepaper updated on the main site?

@@ -1,3 +1,5 @@
![CI](https://github.com/nottoseethesun/Gridcoin-Tasks/workflows/CI/badge.svg)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this linking to your personal repo?

Also the link goes to "no status"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's just my fork.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that perhaps in the over two years since I made this PR, GitHub has changed the Actions such that the old build is no longer here.

@@ -242,7 +242,7 @@ View it like so: The protocol mints a chunk of GRC to distribute to network part

### The Gridcoin Foundation

The Gridcoin Foundation is a community-managed wallet of ~30 million GRC.The Gridcoin
The Gridcoin Foundation is a community-managed wallet of ~30 million GRC. The Gridcoin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is no longer true. The current balance is ~16.6 million GRC

Comment on lines +19 to +20
- name: Checkout the project source code
run: git checkout
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- name: Checkout the project source code
run: git checkout

This should be redundant in the CI.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So there's something else that goes this already?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The actions/checkout action already fetches and checks out the repo. I am actually not sure what a git checkout without branch does.


jobs:
build:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

nit: unnecessary whitespace.

WHITEPAPER_FILE_NAME_ROOT: gridcoin-whitepaper-draft
WHITEPAPER_ARTIFACT_NAME: gridcoin-whitepaper
steps:
- uses: actions/checkout@v2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- uses: actions/checkout@v2
- uses: actions/checkout@v3

Could you update this to v3? v2 leaves warnings in the build logs as it uses some deprecated method.

@@ -230,9 +230,9 @@ MI = Monetary Inflation. There is no ultimate cap on the number of GRC to be min

##### CBR

CBR = Constant Block Reward. CBR incentivizes people to maintain a balance of GRC to help secure the network. There is technical and economic significance to CBR. The economic significance comes in the form of GRC lock-up -- it is similar to how banks reward people with interest for keeping their money in a bank. Essentially, a 10 GRC reward for staking a block (a constant reward for each block) incentivizes people to hold GRC. This type of holding of the GRC helps ensure a healthy network because of the technical ways proof-of-stake works. Similar to how holding money in a bank helps ensure that banks can continue to stimulate economic activity.
CBR = Constant Block Reward. CBR incentivizes people to maintain a balance of GRC to help secure the network. There is technical and economic significance to CBR. The economic significance comes in the form of GRC lock-up -- it is similar to how banks reward people with interest for keeping their money in a bank. Essentially, a 10 GRC reward for staking a block (a constant reward for each block) incentivizes people to hold GRC. This type of holding of the GRC helps ensure a healthy network because of the technical ways proof-of-stake works. The mechanism is similar to how holding money in a bank helps ensure that banks can continue to stimulate economic activity.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
CBR = Constant Block Reward. CBR incentivizes people to maintain a balance of GRC to help secure the network. There is technical and economic significance to CBR. The economic significance comes in the form of GRC lock-up -- it is similar to how banks reward people with interest for keeping their money in a bank. Essentially, a 10 GRC reward for staking a block (a constant reward for each block) incentivizes people to hold GRC. This type of holding of the GRC helps ensure a healthy network because of the technical ways proof-of-stake works. The mechanism is similar to how holding money in a bank helps ensure that banks can continue to stimulate economic activity.
CBR = Constant Block Reward. CBR incentivizes people to maintain a balance of GRC to help secure the network. There is technical and economic significance to CBR. The economic significance comes in the form of GRC lock-up -- it is similar to how banks reward people with interest for keeping their money in a bank. Essentially, a 10 GRC reward for staking a block (a constant reward for each block) incentivizes people to hold GRC. This type of holding of the GRC helps ensure a healthy network because of the technical ways proof-of-stake works. The mechanism is similar to how holding money in a bank helps ensure that banks can continue to stimulate economic activity.

nit: double space.

Slack @todo: this link needs to be fixed
<https://join.slack.com/t/teamgridcoin/shared_invite/enQtMjk2NTI4MzAwMzg0LTUzMmY0YjdiNzYxYzQ0MDc3MGE1NjQ3Nzg2NWMzZTUzMjAwZjdhMWI1YWUzMDE4YzQyZjVjMjMzOTc1MRmMmM>
Slack
<https://join.slack.com/t/teamgridcoin/shared_invite/enQtMjk2NTI4MzAwMzg0LTE4N2I3ZWZjYWJlZGM1Zjg3MTUyMDhiN2M5NmRmZTA2NDA0ZmY1ZTFmOGM3ZGU2YTBkOTdhNTk2ZjkzMGZkODY>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated to this PR but can we show a bit love to Matrix(https://matrix.to/#/#gridcoin-community:matrix.org) and IRC(#gridcoin on irc.libera.chat)? A cryptocurrency which only lists proprietary messaging channels is a bit hypocritical.

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.

5 participants