Plots2: 「人数」で並べ替える場合は、トピックカードの投稿数の前に「-」を削除してください

作成日 2020年05月28日  ·  25コメント  ·  ソース: publiclab/plots2

問題を説明してください

「人数」の並べ替えオプションをクリックすると、トピックカードの投稿数の前に小さな「-」が表示され、投稿数がマイナスになっているように見えます。 スクリーンショットを参照してください:

Screen Shot 2020-05-20 at 10 36 42 AM (1)

何を期待していなかったのですか?

私は正の整数を見ることを期待していました。

どこを見ればいいのか教えてください

https://publiclab.org/tags?sort=followers&order=asc

PublicLab.orgのユーザー名は何ですか?

これは、問題の診断に役立ちます。

リズ

bug

最も参考になるコメント

ちょっと@ebarryこれは実際にはもっと深刻な問題だと思います:sweat_smile:私はここでこれに変更を加えましたhttps://github.com/publiclab/plots2/pull/7609しかし、私はそれを台無しにして間違ったカウントを計算したに違いありませんthink: dispointedいる申し訳ありませんが、これを修正するには助けが必要かもしれません:sweat_smile:

全てのコメント25件

ここに追加できるコードリンクを手元に持っている人はいますか? 前もって感謝します!

ちょっと@ebarryこれは実際にはもっと深刻な問題だと思います:sweat_smile:私はここでこれに変更を加えましたhttps://github.com/publiclab/plots2/pull/7609しかし、私はそれを台無しにして間違ったカウントを計算したに違いありませんthink: dispointedいる申し訳ありませんが、これを修正するには助けが必要かもしれません:sweat_smile:

こんにちは@Tlazypanda Tag.counterタイプコードが使用されている他の場所を見て、それに切り替えてみることができますか?

https://github.com/publiclab/plots2/pull/7609/files#diff -8fe92870a7f027a46eed82a51c5c16bcR118

ただし、カウントはほとんどずれていません。 これはスパムされているものの問題であり、それが数学に少し影響を与えているのではないかと思います。

たとえば、これらのカウントの違いを参照してください。

image

image

236カウントが計算される場所を探して、それに切り替えることができますか?

したがって、コントローラーで生成されたインスタンス変数を使用します。

https://github.com/publiclab/plots2/blob/c056cb0b97ebc6b34184ec3dfa9a83e1096395f9/app/views/tag/show.html.erb#L58

私はこれがこれが計算されるところだと信じています:

https://github.com/publiclab/plots2/blob/c056cb0b97ebc6b34184ec3dfa9a83e1096395f9/app/controllers/tag_controller.rb#L551 -L560

ねえ@jywarren 、私がtag /:idで見ることができるのは、つまりタグの表示ページは、そこにある研究ノートのみを表示していることと、トピックカードに研究ノート+質問を表示しようとしていることです。
したがって、私たちがしなければならないことは、 tag#showで行うのと同じ方法でメモをフェッチする必要があり、トピックカードの下部にカウントを表示するには、次の数式を使用する必要があると思います。
タグ名の付いたメモの数-トピックカードに表示されたメモの数

tag#showで行うのと同じ方法でメモをフェッチする必要があります

こんにちは、それはちょうどいいです! ここで「信頼できる唯一の情報源」の概念に注意してください//github.com/publiclab/plots2/issues/6855#issuecomment -578331475

ここでも同じ計算を使用する必要があると思います: https

@ urvashigupta7チャイムをしてくれてありがとう! これらの変更を行うことに興味がありますか?

また、 https://github.com/publiclab/plots2/pull/7476/では、スパムを除外していないと思います。 したがって、これらの行に、おそらく.where(status: 1)追加する必要があります。

https://github.com/publiclab/plots2/blob/c056cb0b97ebc6b34184ec3dfa9a83e1096395f9/app/controllers/tag_controller.rb#L551 -L560

そのコードもこれに依存しており、スパムをフィルタリングしません。

https://github.com/publiclab/plots2/blob/f4dfdcbae4dfa60b0ec357081ee1fae6f2089436/app/models/node.rb#L609 -L618

同様に、このコードも同じフィルターでスパムをフィルターで除去する必要があります。

https://github.com/publiclab/plots2/blob/f4dfdcbae4dfa60b0ec357081ee1fae6f2089436/app/models/tag.rb#L118 -L124

うーん。 次の行の/ after /を実行するため、 @total_posts計算からスパムをフィルタリングしているようです。

https://github.com/publiclab/plots2/blob/c056cb0b97ebc6b34184ec3dfa9a83e1096395f9/app/controllers/tag_controller.rb#L204 -L218

ただし、 Tag.counterNode.for_tagname_and_typeどちらもフィルタリングしません。 多分それは不規則ですか?

ここでのカウントはまだ正しくないと思います。 たとえば、 https ://stable.publiclab.org/tagsはballoon-mapping 692件の投稿を一覧表示しますが、そのタグの統計は次のとおりです(安定版)。

image

この修正に依存する必要があると思います: https

その後、これをもう一度確認できます。 しばらく再開します...指🤞

https://github.com/publiclab/plots2/pull/8245が完了しました! そして、ユニットテストがあるので、とてもいいです。 OK、これでうまくいくかもしれません...

OK、 573 posts » -うーん

研究ノート469
質問37
Wikiページ77
人々346

つまり、メモとウィキ= 546 ...? プラスの質問は583ですか?

さて、stableでタグtestsをテストしていると、キャッシュが行われているのではないかと疑っています。 そのタグを新しいメモhttps://stable.publiclab.org/notes/aidanswope/05-21-2019/angel-s-point-balloon-map-updatedに追加しても、メモのタブはtag.run_count結果と同様に、 tag.countは8を返しますが、7から8です。

https://stable.publiclab.org/tag/tests

https://github.com/publiclab/plots2/issues/8244へのリンクに注意して

YESSSSS 5分間キャッシュされます: https

OK、タブ数をトレースします。

https://github.com/publiclab/plots2/blob/e97eff7464a544a97f4b38f169f76250600b3148/app/models/node.rb#L621 -L630

Node.where(status: 1, type: 'note').includes(:revision, :tag).references(:term_data, :node_revisions).where('term_data.name = ? OR term_data.parent = ?', 'balloon-mapping', 'balloon-mapping').where.not(nid: qids).count => 470

トピックカード数との比較:

NodeTag.joins(:node).where(tid: 10).where('node.status = 1 AND node.type = "note"').where.not(nid: qids).count => 467

😭

これらの3つが違いです: [6129, 7236, 7414] -

[#

]

非常に奇妙な! 2つの理由:3つのメモは10の欠落しているメモと同じではありません(まだ質問を調べていないと思います)。また、これらの3つは、1つが2013年2月で、2つが2013年5月であったことを除いて正常のようです。

(更新:重複するタグ名については以下を参照してください)

また興味深い:

Node.where(status: 1, type: 'note').includes(:revision, :tag).references(:term_data, :node_revisions).where('term_data.name = ? OR term_data.parent = ?', 'balloon-mapping', 'balloon-mapping').where(nid: qids).count => 29

これは次のことに同意します。

NodeTag.joins(:node).where(tid: 10).where('node.status = 1 AND node.type = "note"').where(nid: qids).count

対37、タブに示されているように、次のように計算されます。

https://github.com/publiclab/plots2/blob/e97eff7464a544a97f4b38f169f76250600b3148/app/models/node.rb#L640 -L646

あはは! その最後のセクションは、ベースのballoon-mappingタグがなくても、 question:balloon-mappingタグが付けられた投稿がそのカウントに含まれることを示しています!!!

次の8つの投稿にはquestion:balloon-mappingがあるようですが、 balloon-mappingはありません:

[13556, 15762, 16426, 18994, 20694, 11301, 12251, 19625]

OK、これで2ノード以内に到達します。 それでは、手動で比較してみましょう。

irb(main):040:0> NodeTag.joins(:node).where(tid: 10).where('node.status = 1 AND node.type = "note"').count
=> 496
irb(main):041:0> Node.where(status: 1, type: 'note').includes(:tag).references(:term_data).where('term_data.name = ?', 'balloon-mapping').count
=> 499
irb(main):022:0> Node.where(status: 1, type: 'note').includes(:revision, :tag).references(:term_data, :node_revisions).where('term_data.name = ? OR term_data.parent = ?', 'balloon-mapping', 'balloon-mapping').collect(&:nid)
=> [146, 253, 254, 268, 403, 511, 724, 732, 743, 913, 1482, 1489, 1620, 1626, 1627, 1632, 1715, 1849, 1968, 1994, 2126, 2138, 2144, 2476, 2589, 2930, 3115, 3325, 4649, 4952, 5038, 5220, 5250, 5978, 6934, 7050, 7189, 7294, 7314, 7491, 7577, 7593, 7995, 8189, 8907, 8913, 8914, 8950, 9019, 9032, 9042, 9086, 9417, 9434, 9742, 9888, 9977, 10541, 11019, 11100, 11223, 11574, 11582, 11827, 12201, 12240, 12491, 12619, 12708, 13095, 13424, 13945, 14022, 14095, 14183, 14185, 14219, 14515, 14572, 14587, 14597, 14600, 14611, 14620, 14813, 14907, 14909, 14983, 15545, 15568, 16591, 16653, 16746, 16877, 18316, 19153, 19272, 19462, 19558, 19578, 19620, 19762, 20331, 23549, 24101, 24102, 1795, 6635, 347, 369, 439, 444, 792, 861, 1665, 1668, 1962, 2135, 2143, 2152, 2245, 2277, 2783, 2784, 3563, 3635, 3670, 4482, 4516, 4755, 4760, 4923, 4925, 6059, 7641, 10181, 10489, 11101, 11724, 13243, 13880, 13948, 17322, 64, 166, 202, 344, 1666, 1816, 1818, 1819, 2919, 3354, 3736, 3821, 4087, 8020, 8947, 9301, 11199, 11260, 11261, 11755, 12161, 1544, 1739, 1742, 1892, 2006, 5494, 9353, 10705, 828, 1515, 1786, 1877, 1926, 1949, 2121, 2625, 2666, 2677, 2846, 2892, 2961, 4092, 4907, 5394, 5415, 5664, 7132, 7367, 7368, 7717, 7964, 8400, 10243, 6175, 9507, 4636, 6293, 3014, 6624, 9806, 10289, 10554, 11278, 11391, 12104, 12622, 12894, 13143, 9935, 10036, 10196, 10450, 11283, 11719, 11769, 12237, 10246, 10605, 2462, 9497, 1889, 2131, 2443, 3026, 4127, 4252, 4673, 5538, 5893, 6608, 8034, 8902, 8959, 9644, 9707, 10045, 10162, 10335, 10336, 10337, 10373, 10540, 10570, 11107, 11110, 11112, 11124, 11143, 11175, 11298, 11336, 12053, 12203, 12590, 16984, 19515, 19580, 23822, 2163, 3634, 7620, 10166, 11056, 11319, 11589, 1541, 1652, 1676, 1677, 1746, 2154, 2237, 2275, 2311, 2623, 2785, 3454, 3652, 3822, 4014, 4691, 4720, 4950, 5161, 7785, 8866, 8926, 8928, 8937, 8958, 8986, 9759, 9915, 10101, 10230, 10244, 10302, 10311, 10374, 10523, 10602, 11021, 11078, 11103, 11253, 11726, 11988, 12686, 12752, 12757, 13400, 13845, 14116, 14660, 14713, 14756, 1716, 2660, 7191, 7192, 1743, 1636, 1900, 9317, 10651, 1741, 13904, 3096, 2352, 2824, 4091, 2715, 6080, 6082, 2732, 3801, 3908, 4545, 7121, 10284, 13791, 10631, 4921, 6751, 9429, 9743, 11577, 17331, 6073, 9858, 10711, 11118, 6525, 10267, 9034, 11252, 11268, 8033, 8868, 10159, 10214, 9027, 9785, 9033, 9076, 9299, 14501, 14508, 14562, 21733, 10282, 9909, 11178, 11738, 10024, 12667, 11262, 11343, 11978, 11983, 11992, 11826, 11061, 10227, 10428, 11077, 11131, 11137, 11280, 11281, 11518, 11551, 11807, 11812, 11947, 12039, 14158, 14729, 14943, 15641, 16273, 16349, 18560, 19269, 19286, 21547, 10343, 14852, 14901, 15352, 15387, 10612, 10638, 10986, 10994, 20181, 11002, 11127, 11128, 11183, 12166, 11342, 11494, 11955, 11673, 11707, 11711, 12190, 18839, 13802, 14595, 12227, 15912, 14007, 13093, 13096, 12981, 13102, 13160, 13194, 13196, 13227, 13242, 13267, 13278, 13281, 13298, 14168, 15161, 23143, 14058, 14766, 13748, 13839, 15603, 14241, 14179, 14239, 14246, 14240, 14255, 14488, 14487, 14245, 14785, 15160, 14703, 14995, 16571, 15172, 15179, 15424, 15713, 15732, 16141, 16284, 16334, 16382, 17387, 17498, 17499, 18498, 16960, 19689, 17815, 19394, 19464, 19650, 19955, 19476, 19477, 23203, 6129, 7236, 7414]

対:


irb(main):024:0> NodeTag.joins(:node).where(tid: 10).where('node.status = 1 AND node.type = "note"').collect(&:nid)
=> [146, 253, 254, 268, 403, 511, 724, 732, 743, 913, 1482, 1489, 1620, 1626, 1627, 1632, 1715, 1849, 1968, 1994, 2126, 2138, 2144, 2476, 2589, 2930, 3115, 3325, 4649, 4952, 5038, 5220, 5250, 5978, 6934, 7050, 7189, 7294, 7314, 7491, 7577, 7593, 7995, 8189, 8907, 8913, 8914, 8950, 9019, 9032, 9042, 9086, 9417, 9434, 9742, 9888, 9977, 10541, 11019, 11100, 11223, 11574, 11582, 11827, 12201, 12240, 12491, 12619, 12708, 13095, 13424, 13945, 14022, 14095, 14183, 14185, 14219, 14515, 14572, 14587, 14597, 14600, 14611, 14620, 14813, 14907, 14909, 14983, 15545, 15568, 16591, 16653, 16746, 16877, 18316, 19153, 19272, 19462, 19558, 19578, 19620, 19762, 20331, 23549, 24101, 24102, 1795, 6635, 347, 369, 439, 444, 792, 861, 1665, 1668, 1962, 2135, 2143, 2152, 2245, 2277, 2783, 2784, 3563, 3635, 3670, 4482, 4516, 4755, 4760, 4923, 4925, 6059, 7641, 10181, 10489, 11101, 11724, 13243, 13880, 13948, 17322, 64, 166, 202, 344, 1666, 1816, 1818, 1819, 2919, 3354, 3736, 3821, 4087, 8020, 8947, 9301, 11199, 11260, 11261, 11755, 12161, 1544, 1739, 1742, 1892, 2006, 5494, 9353, 10705, 828, 1515, 1786, 1877, 1926, 1949, 2121, 2625, 2666, 2677, 2846, 2892, 2961, 4092, 4907, 5394, 5415, 5664, 7132, 7367, 7368, 7717, 7964, 8400, 10243, 6175, 9507, 4636, 6293, 3014, 6624, 9806, 10289, 10554, 11278, 11391, 12104, 12622, 12894, 13143, 9935, 10036, 10196, 10450, 11283, 11719, 11769, 12237, 10246, 10605, 2462, 9497, 1889, 2131, 2443, 3026, 4127, 4252, 4673, 5538, 5893, 6608, 8034, 8902, 8959, 9644, 9707, 10045, 10162, 10335, 10336, 10337, 10373, 10540, 10570, 11107, 11110, 11112, 11124, 11143, 11175, 11298, 11336, 12053, 12203, 12590, 16984, 19515, 19580, 23822, 2163, 3634, 7620, 10166, 11056, 11319, 11589, 1541, 1652, 1676, 1677, 1746, 2154, 2237, 2275, 2311, 2623, 2785, 3454, 3652, 3822, 4014, 4691, 4720, 4950, 5161, 7785, 8866, 8926, 8928, 8937, 8958, 8986, 9759, 9915, 10101, 10230, 10244, 10302, 10311, 10374, 10523, 10602, 11021, 11078, 11103, 11253, 11726, 11988, 12686, 12752, 12757, 13400, 13845, 14116, 14660, 14713, 14756, 1716, 2660, 7191, 7192, 1743, 1636, 1900, 9317, 10651, 1741, 13904, 3096, 2352, 2824, 4091, 2715, 6080, 6082, 2732, 3801, 3908, 4545, 7121, 10284, 13791, 10631, 4921, 6751, 9429, 9743, 11577, 17331, 6073, 9858, 10711, 11118, 6525, 10267, 9034, 11252, 11268, 8033, 8868, 10159, 10214, 9027, 9785, 9033, 9076, 9299, 14501, 14508, 14562, 21733, 10282, 9909, 11178, 11738, 10024, 12667, 11262, 11343, 11978, 11983, 11992, 11826, 11061, 10227, 10428, 11077, 11131, 11137, 11280, 11281, 11518, 11551, 11807, 11812, 11947, 12039, 14158, 14729, 14943, 15641, 16273, 16349, 18560, 19269, 19286, 21547, 10343, 14852, 14901, 15352, 15387, 10612, 10638, 10986, 10994, 20181, 11002, 11127, 11128, 11183, 12166, 11342, 11494, 11955, 11673, 11707, 11711, 12190, 18839, 13802, 14595, 12227, 15912, 14007, 13093, 13096, 12981, 13102, 13160, 13194, 13196, 13227, 13242, 13267, 13278, 13281, 13298, 14168, 15161, 23143, 14058, 14766, 13748, 13839, 15603, 14241, 14179, 14239, 14246, 14240, 14255, 14488, 14487, 14245, 14785, 15160, 14703, 14995, 16571, 15172, 15179, 15424, 15713, 15732, 16141, 16284, 16334, 16382, 17387, 17498, 17499, 18498, 16960, 19689, 17815, 19394, 19464, 19650, 19955, 19476, 19477, 23203]

あは。 多分これ:

Tag.where(name: 'balloon-mapping').length => 4
Tag.where(name: 'balloon-mapping').collect(&:tid) => [10, 3205, 3591, 4032]

はい、それは矛盾です:

irb(main):045:0> NodeTag.joins(:node).where(tid: [10, 3205, 3591, 4032]).where('node.status = 1 AND node.type = "note"').count
=> 499
irb(main):046:0> Node.where(status: 1, type: 'note').includes(:tag).references(:term_data).where('term_data.name = ?', 'balloon-mapping').count
=> 499

irb(main):050:0> Tag.where(name: 'balloon-mapping').collect(&:count)
=> [574, 1, 1, 1]

ただし、特に#7334のmethodまたはprojectバグは対象外です。

irb(main):047:0> Tag.where(name: 'method').collect(&:tid)
=> [14474]
irb(main):048:0> Tag.where(name: 'project').collect(&:tid)
=> [14751]

OK、重複するタグを次のように修正しました:

irb(main):053:0> NodeTag.where(tid: [3205, 3591, 4032]).each do |nt|
irb(main):054:1* nt.tid = 10
irb(main):055:1> nt.save
irb(main):056:1> end

しかし、私はDBにもっとあるに違いない。 これらを統合するために、より大きなプロジェクトを実行したい場合があります。

更新:はい、他に28のインスタンスがあります:

irb(main):063:0> Tag.count
=> 18532
irb(main):065:0> Tag.all.collect(&:name).uniq.count
=> 18504


irb(main):069:0> (Tag.all - Tag.all.group(:name)).collect(&:name)
=> ["fold-up", "gulf-coast", "kite-mapping", "gulf-coast", "brooklyn", "new-york-city", "balloon-mapping", "spectrometer", "farmhack", "white-balance", "fresh-kills", "balloon-mapping", "balloon-mapping", "kite-mapping", "newsletter", "newsletter", "passenger-pigeon", "jamaica-bay", "mobile-spectrometer", "infragram-sandbox", "mailing-lists", "diy-kite", "water-sensing", "carbon-arc", "diy-kite", "diy-kite", "workplace", "emergencies"]

今の私の推奨事項は次のとおりです。

これをかなり絞り込んで、2つの部分に分かれています。

  1. 「ゴースト」タグ-タグ名が重複しているため、基本的に一部はカウントされませんでした(解決済みですが、他のいくつかのタグには最大15個のインスタンスが存在します)
  2. question:_____ /tag/____ページの[質問]タブでカウントされるquestion:_____形式のタグですが、独自のベースタグがない場合は、トピックカードにカウントされません。 例:8件の投稿にはquestion:balloon-mappingがありますが、 balloon-mappingはないため、カウントは8になります。

(2)についての私の見解は、それを書き留めて認識すべきであるが、それを解決するべきではないということです(とにかくクリーンな解決策を考えることはできません)。

そして、そのすべての掘り下げの後でも、私はまだballoon-mappingに対して1の不一致があります。 しかし、これは最大のタグの1つであり、その問題が他の人に影響を与える可能性は低いため、私たちは非常に近いので、基本的には問題ないと思います。 他の人の意見も聞いてみたいです!

このページは役に立ちましたか?
0 / 5 - 0 評価