holgern / beem

A python library to interact with the HIVE blockchain
http://beem.readthedocs.io/en/latest/
Other
135 stars 73 forks source link

Discussions.get_discussions("blog", ...) returns the same two comments over and over #51

Closed crokkon closed 6 years ago

crokkon commented 6 years ago

Expected behavior

Discussions.get_discussions('blog', query) should return a number of blog posts from a steem account based on the query parameter.

Actual behavior

Discussions.get_discussions('blog', query) may return the same two posts over and over

How to reproduce

from beem.discussions import Query, Discussions
start_author = "crokkon"
start_permlink = "how-to-find-out-which-of-your-steem-apps-just-acted-in-your-name"
query_limit = 20
query = Query(start_author=start_author,
              start_permlink=start_permlink, limit=query_limit,
              tag=start_author)
dis = Discussions()
count = 0
for d in dis.get_discussions("blog", query):
    print(("%d. " % (count + 1)) + str(d))
    count += 1

Output:

1. <Comment @crokkon/how-to-find-out-which-of-your-steem-apps-just-acted-in-your-name>
2. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
3. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
4. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
5. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
6. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
7. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
8. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
9. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
10. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
11. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
12. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
13. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
14. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
15. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
16. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
17. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
18. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
19. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
20. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
...

Environment

holgern commented 6 years ago

Thanks, issue is fixed in https://github.com/holgern/beem/commit/041647798c80a499e2b82652050f9e88c156d937

The result is now

1. <Comment @crokkon/one-key-to-rule-them-all-analysis-of-key-resuage-across-steem-accounts>
2. <Comment @crokkon/the-busy-org-free-upvote-bot-adoption-by-users-and-performance-jan-2018>
3. <Comment @crokkon/plagiarists-tricking-cheetah-by-replacing-chars-with-lookalike-unicode-symbols>
4. <Comment @crokkon/signatures-of-early-and-late-bid-bot-boosted-posts-and-possible-abuse-counter-measures>
5. <Comment @crokkon/effective-curation-analysis-of-curation-rewards-in-theory>
6. <Comment @crokkon/steemit-is-taking-its-delegated-sp-back-analysis-of-recent-steem-un-delegations>
7. <Comment @crokkon/the-busy-org-free-upvote-bot-adoption-by-users-and-performance-since-nov-2017>
8. <Comment @crokkon/can-curation-be-automated-and-is-it-good-for-steem-the-curx-curation-bot-experiment>
9. <Comment @crokkon/curation-rewards-you-could-have-had-calculating-the-best-voting-time-sweet-spot-retrospectively>
10. <Comment @crokkon/don-t-use-steem-keys-as-transfer-memos-analysis-of-private-and-public-keys-in-transfer-memos-and-implications-to-encrypted-memos>
11. <Comment @crokkon/utopian-created-github-issue-twice>
12. <Comment @crokkon/curation-reward-estimation-slight-deviations-in-the-calculation-of-rewards>
13. <Comment @crokkon/gitlocsvg-git-repository-lines-of-code-evolution-graph-as-vector-graphic-svg>
14. <Comment @crokkon/cashout-chains-part-ii-power-down-and-cashout-of-initial-account-sp>
15. <Comment @paulag/business-intelligence-steemit-weekly-contest-11-and-winner-10>
16. <Comment @crokkon/cashout-chains-steem-from-7400-accounts-leaves-steem-via-only-a-few-accounts>
17. <Comment @crokkon/using-the-utopian-api-interface-for-bot-analysis-bisteemit>
18. <Comment @crokkon/sbd-transfer-memos-categories-urls-and-private-steem-keys>
19. <Comment @crokkon/a-glimpse-into-a-news-bot-network>
20. <Comment @paulag/business-intelligence-steemit-weekly-contest-9-and-winner-8>
21. <Comment @crokkon/accessing-the-steem-blockchain-via-mongodb-and-python-with-any-linux-distribution-using-docker>
22. <Comment @crokkon/utopian-bot-qurator-bot-is-present-twice-in-the-list-of-known-bots>
23. <Comment @crokkon/utopian-io-bot-analysis-of-median-vs-average-body-sizes>
24. <Comment @crokkon/webgui-expected-utopian-post-parameters>
25. <Comment @crokkon/market-share-of-alternative-steem-apps-08-10-17>
26. <Comment @paulag/business-intelligence-steemit-weekly-contest-8-and-winner-7>
27. <Comment @crokkon/utopian-getting-grip-usage-analysis-of-the-first-weeks>
28. <Comment @crokkon/minnowbooster-performance-analysis-aug-oct-2017>
29. <Comment @crokkon/minnowbooster-s-sell-your-vote-from-blockchain-perspective>
30. <Comment @crokkon/blockchain-analytics-3-upvote-bot-vote-intervals>
31. <Comment @crokkon/steem-blockchain-access-with-python-setting-up-steem-python-with-docker>
32. <Comment @crokkon/piston-quickstart>
33. <Comment @crokkon/blockchain-analytics-2-bellyrub-s-rubrubrubbing-statistics-sept-2017>
34. <Comment @crokkon/blockchain-analytics-1-jerrybanfield-s-voting-behavior>
35. <Comment @macbaren/giveaway-i-will-give-usd2-steem-dollars-just-for-upvoting-and-resteeming-plus-extra>
36. <Comment @crokkon/using-piston-with-a-local-steem-blockchain>
37. <Comment @crokkon/how-to-hide-resteems-from-certain-users-with-your-adblocker>
38. <Comment @crokkon/steem-blockchain-size>
39. <Comment @crokkon/resteems-flooding-your-home-feed-simply-hide-them>
40. <Comment @crokkon/hello-steemit>
41. <Comment @crokkon/hard-fork-preparations-new-monero-release-helium-hydra>
42. <Comment @crokkon/hide-your-mining-activities-with-tor>
43. <Comment @unbeaten/bitcoin-hardware-wallet-ledger-begins-monero-integration>
44. <Comment @barton26/new-boinc-version-7-8-2-released>
45. <Comment @crokkon/firefox-and-alsa-sound-without-pulseaudio>
46. <Comment @crokkon/give-me-15-minutes-and-i-ll-change-your-view-of-gdb-greg-law-cppcon15>
47. <Comment @crokkon/spectrecoin-a-new-privacy-coin-on-the-rise>
48. <Comment @crokkon/monero-hard-fork-september-2017-what-changes>
49. <Comment @biophil/genesis-pocket>
50. <Comment @crokkon/the-definitive-crypto-guide-for-beginners-and-veterans-alike>
51. <Comment @cryptores/why-monero-xmr-is-not-only-a-currency-for-the-dark-web-why-privacy-matters-and-why-anonymous-currencies-are-on-the-rise>
52. <Comment @cryptounicorn/i-made-usd10-000-in-crypto-the-last-month-without-investing-a-single-penny-crying-tears-of-joy>

The output stops at entry 52, as the RPC node does not return more data.

crokkon commented 6 years ago

awesome! :)