mimblewimble/grin

Update mining status code to support multiple "edge_bits" value

Ouverte

#1 875 ouverte le 29 oct. 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Rust (991 forks)batch import
enhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (4 876 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Examinez le code TUI de minage dans grin miner pour comprendre comment la valeur GPS actuelle est affichée. Identifiez les sources de données pour edge bits et les taux de graphe de différents plugins. Proposez des modifications pour normaliser ou afficher des valeurs GPS séparées. Envisagez la mise en œuvre de l'option 2 (afficher chaque taux séparément) ou une vue combinée.
Stack technique
rust
Domaine
backendcliblockchain
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
RustMining concepts
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50