mimblewimble/grin

Update mining status code to support multiple "edge_bits" value

Open

#1.875 geöffnet am 29. Okt. 2018

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (991 Forks)batch import
enhancementgood first issuehelp wanted

Repository-Metriken

Stars
 (4.876 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 11h) (25 gemergte PRs in 30 T)

Beschreibung

Now that we support multiple cuckatoo/cuckaroo sizes having a single "gps" value displayed may be confusing.

The grin-miner tui says "Mining status: Mining at height xxx at yyy GPS

If the user is mining with 2 plugins, a CPU mining edge_bits=29, and a GPU mining edge_bits=30, that message with a single GPS means what?

I see two options (but a third better option may exist):

  1. normalize and merge all graph rates into a single GPS
  2. display each rate as a separate GPS value

Contributor Guide