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

feat: added generate-nano-reps script #84

Merged
merged 11 commits into from
Mar 25, 2024
Merged

Conversation

mistakia
Copy link
Owner

@mistakia mistakia commented Mar 2, 2024

Created a script to reconcile data in the nano.community database with the csv file at data/nano-reps.csv. The data in data/nano-reps.csv will serve as a source of truth moving forward.

The only conflicting field I didn't know how to reconcile. Perhaps we can contact them and see what their preference is.

  generate-nano-reps conflicting edits for account: nano_3chartsi6ja8ay1qq9xg3xegqnbg1qx76nouw6jedyb8wx3r4wu94rxap7hg +0ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'twitter' ],
  generate-nano-reps     lhs: 'NanoCharts',
  generate-nano-reps     rhs: '@kilkelly'
  generate-nano-reps   }
  generate-nano-reps ] +0ms

I've also opened a new issue to help ensure discord username freshness: #83

@mistakia mistakia requested a review from qwahzi March 2, 2024 05:25
@mistakia mistakia added need/triage Needs initial labeling and prioritization priority/medium Medium: Good to have, but can wait until someone steps up kind/maintenance Work required to avoid breaking changes or harm to project's status quo status/in-progress In progress and removed need/triage Needs initial labeling and prioritization labels Mar 2, 2024
@mistakia mistakia force-pushed the feat/generate-nano-reps branch 2 times, most recently from 87382f6 to cfc09f4 Compare March 3, 2024 03:57
@mistakia
Copy link
Owner Author

mistakia commented Mar 3, 2024

Need to resolve the following conflicts:

  generate-nano-reps conflicting edits for account: nano_34amtofxstsfyqcgphp8piij9u33widykq9wbz6ysjpxhbgmqu8btu1eexer +0ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'reddit' ],
  generate-nano-reps     lhs: 'tucsonthrowaway3',
  generate-nano-reps     rhs: 'Roconda'
  generate-nano-reps   }
  generate-nano-reps ] +1ms
  generate-nano-reps conflicting edits for account: nano_3zapp5z141qpjipsb1jnjdmk49jwqy58i6u6wnyrh6x7woajeyme85shxewt +4ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'reddit' ],
  generate-nano-reps     lhs: 'Laughingboy14',
  generate-nano-reps     rhs: 'numsu'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_3chartsi6ja8ay1qq9xg3xegqnbg1qx76nouw6jedyb8wx3r4wu94rxap7hg +1ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'twitter' ],
  generate-nano-reps     lhs: 'NanoCharts',
  generate-nano-reps     rhs: 'kilkelly'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_34zuxqdsucurhjrmpc4aixzbgaa4wjzz6bn5ryn56emc9tmd3pnxjoxfzyb6 +0ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'reddit' ],
  generate-nano-reps     lhs: 'Jan-d',
  generate-nano-reps     rhs: 'jan-d'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_1fe17w13stn8rqos3nxmupoez9sne4pc4njmr1fbz9nci6obnng6jatton5q +1ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'discord' ],
  generate-nano-reps     lhs: 'imp3lor',
  generate-nano-reps     rhs: '_derjan_'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_3power3gwb43rs7u9ky3rsjp6fojftejceexfkf845sfczyue4q3r1hfpr3o +1ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'alias' ],
  generate-nano-reps     lhs: 'PowerNode',
  generate-nano-reps     rhs: "NF - The architect of Nano's ruin.  // nanotps.com"
  generate-nano-reps   }
  generate-nano-reps ] +0ms

Currently, only includes reps with 1000 Nano voting weight which includes some reps with no alias or known fields. Thoughts? @qwahzi

I wonder if it should be a requirement where at least one other field is known to be included.

@mistakia mistakia force-pushed the feat/generate-nano-reps branch from 7d63a0e to a002f7e Compare March 4, 2024 02:45
@mistakia mistakia force-pushed the feat/generate-nano-reps branch from a002f7e to 20e0941 Compare March 25, 2024 00:38
@mistakia
Copy link
Owner Author

mistakia commented Mar 25, 2024

I have reached out to those operators and requested for clarification on these remaining conflicts

  generate-nano-reps conflicting edits for account: nano_34amtofxstsfyqcgphp8piij9u33widykq9wbz6ysjpxhbgmqu8btu1eexer +0ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'reddit' ],
  generate-nano-reps     lhs: 'tucsonthrowaway3',
  generate-nano-reps     rhs: 'Roconda'
  generate-nano-reps   }
  generate-nano-reps ] +1ms
  generate-nano-reps conflicting edits for account: nano_3zapp5z141qpjipsb1jnjdmk49jwqy58i6u6wnyrh6x7woajeyme85shxewt +4ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'reddit' ],
  generate-nano-reps     lhs: 'Laughingboy14',
  generate-nano-reps     rhs: 'numsu'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_3chartsi6ja8ay1qq9xg3xegqnbg1qx76nouw6jedyb8wx3r4wu94rxap7hg +0ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'twitter' ],
  generate-nano-reps     lhs: 'NanoCharts',
  generate-nano-reps     rhs: 'kilkelly'
  generate-nano-reps   }
  generate-nano-reps ] +0ms
  generate-nano-reps conflicting edits for account: nano_3power3gwb43rs7u9ky3rsjp6fojftejceexfkf845sfczyue4q3r1hfpr3o +1ms
  generate-nano-reps [
  generate-nano-reps   DiffEdit {
  generate-nano-reps     kind: 'E',
  generate-nano-reps     path: [ 'alias' ],
  generate-nano-reps     lhs: 'PowerNode',
  generate-nano-reps     rhs: "NF - The architect of Nano's ruin.  // nanotps.com"
  generate-nano-reps   }
  generate-nano-reps ] +0ms

@mistakia mistakia force-pushed the feat/generate-nano-reps branch from 20e0941 to d4d7f49 Compare March 25, 2024 01:21
@mistakia mistakia force-pushed the feat/generate-nano-reps branch from d4d7f49 to fb4dd52 Compare March 25, 2024 14:49
@mistakia mistakia merged commit 8d5c877 into main Mar 25, 2024
1 check passed
@mistakia mistakia deleted the feat/generate-nano-reps branch March 25, 2024 16:04
ATXMJ pushed a commit to ATXMJ/nano-community that referenced this pull request Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/maintenance Work required to avoid breaking changes or harm to project's status quo priority/medium Medium: Good to have, but can wait until someone steps up status/in-progress In progress
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

1 participant