reubenyap
12 أغسطس 2024، 9:53ص
1
لدينا استطلاع تصويت تفضيلي هنا: [POLL] Final Tokenomics and Block Division - Polls - The Firo Forum
نحن نسمح للناس بتغيير أصواتهم حتى تاريخ الإغلاق.
ومع ذلك، عندما نغير أصواتنا، فإنها لا تنعكس في علامة التبويب النتائج/التصويت. لا يمكنني العثور على طريقة لتصدير البيانات من الاستطلاع على الرغم من أنني مسؤول.
@merefield FIX: poll ranked choice result algo majority check by merefield · Pull Request #28191 · discourse/discourse · GitHub .
إعجابَين (2)
مرحباً روبن، سأرسل لك رسالة خاصة لمناقشة المزيد من التفاصيل.
إعجاب واحد (1)
يبدو أن روبن لم يقم بتطبيق الإصلاح الأخير بطريقة ما، ويبدو أن نتيجة استطلاعه قد تم حلها بشكل صحيح بعد التحديث.
كما نصحته بإضافة المكون الإضافي لاستكشاف البيانات وإلا فلن يكون هناك خيار تصدير. (هذا طبيعي)
ومع ذلك، وجدنا مشكلة ذات صلة بتوسيع قائمة المصوتين في النتائج (عندما يتجاوز عددهم 25) للاختيار من بينها (المشكلة لا توجد في الاستطلاعات القياسية)
الإصلاح هنا:
main ← merefield:fix_ranked_choice_voters_expand
opened 10:58AM - 12 Aug 24 UTC
initially, the list of voters shown on the poll results has a limit of 25 per op… tion.
If the actual number of voters is longer, a button is presented to allow the user to load another page of voters if they so choose.
This calls a method to retrieve the extra voter information.
a bug existed such that the local tracked object was not correctly updated for ranked choice voters as there are two attributes per option, one which tells the interface if new data is currently loading, and the other contains the list of voters.
so instead of updating the entire option key with the voters list, we must update the voters attribute only.
a small but critical change.
The PR includes a significant new test which will add coverage over and above this issue.
@reubenyap تم دمج هذا الآن، يرجى تحديث مثيلك ويجب أن يكون الإصلاح موجودًا. يرجى التأكيد.
مرحباً، بعد التحديث، لا يزال الأشخاص الذين يغيرون أصواتهم غير ظاهرين.
النقر على زر “المزيد/توسيع” أسفل كل تصويت يعرضه فقط.
ثم يعود إلى ما كان عليه سابقًا (SS1) بعد بضع ثوانٍ.
ajaydono:
مرحباً، بعد التحديث
هل يمكنك مشاركة التزام ديسكورس الذي أنت عليه، من لوحة التحكم؟
حسنًا، صدق أو لا تصدق، الإصلاح حسّن الأمور بالفعل، لكن هذا يحتاج إلى عمل إضافي لمنع “إعادة تعيين” التحديثات، وهو ما يحدث في بيئة الإنتاج أسرع مما يحدث في بيئة التطوير.
نحتاج إلى التفكير بعناية في كيفية التعامل مع التحديثات الواردة بعد توسيع قائمة المصوتين.
أنا أميل إلى تجميد قائمة المصوتين من التحديثات البعيدة بمجرد النقر على الزر أو تعيين مهلة زمنية للسماح لك بفحص قائمة المصوتين لفترة محدودة قبل قبول التحديثات من الخادم.
شكراً لصبرك.
إعجاب واحد (1)
reubenyap
13 أغسطس 2024، 1:52م
10
تتغير قائمة الناخبين بشكل كبير على الرغم من أن هذا يتجاوز ما هو عليه تغيير التصويت العادي. أعتقد أن هناك شيئًا خاطئًا.
قبل النقر
إعجاب واحد (1)
reubenyap
13 أغسطس 2024، 1:53م
11
بعد النقر على جميع الأزرار: انظر كيف أنه تحول هائل ويختفي الناس.
إعجاب واحد (1)
تم تقديم طلب السحب الإضافي:
main ← merefield:fix_poll_voter_list_expansion
opened 09:32PM - 13 Aug 24 UTC
An additional follow-up change on top of #28315 (which is still a valid improvem… ent) is required as currently expanded voter lists are almost immediately overwritten by incoming data on the message bus and post stream and effectively cancels the expansion leading to an uncomfortable user experience. This was unfortunately not picked up during unit testing nor by manual testing because in dev the refresh cycle is so long.
So to prevent this and allow a user time to examine the expanded voter list, this change renders incoming updates for voter user lists disabled as soon as any option's voters list is expanded. This only impacts big polls with > 25 voters on an option,
I believe this is a reasonable compromise and it still allows polls with fewer than 26 votes on an option to continue to be streamed updates to voter population as this change does not affect that scale of poll as there is never any expansion action.
Background:
`preloadedVoters` is a list of all users who have voted. It is maintained locally on the client because the data can change from incoming updates from remote voting events _and_ from local events, such as voting or expansion of the collapsed list of voters.
In order to save resources, the server by default will initially only serialize the first 25 voters (per option) down to the client. If the user wishes to see more, they hit the provided expand button. This will pull down an additional page of voters. There are therefore two paths for update and the server does not keep track of the state of the local user interface.
More on the fix:
A user can restore the update process by navigating away from the Topic and returning or refreshing the browser (and that will revert the expanded list to the collapsed view - no change in that behaviour).
It obviously has no downside on Closed polls.
Test updated to reflect the change.
3 إعجابات
reubenyap
16 أغسطس 2024، 4:54م
13
فقط للتحقق، هل من المتوقع أنه قبل توسيع تصويتك، لا ترى تصويتك في الأصوات المقدمة إذا كنت قد غيرت تصويتك في وقت ما من قبل؟
على سبيل المثال، لقد غيرت تصويتي قبل يومين، ولكن عندما أعود إلى الاستطلاع، يظهر تصويتي في العرض على الرغم من أنه يظهر في البيانات الأولية.
مرحباً @reubenyap - لم يكن من المنطقي التحقيق أكثر حتى تم دمج هذا الإصلاح.
لقد تم دمجه الآن.
يجب أن ترى تحسنًا كبيرًا.
إذا كانت هناك مشكلة متبقية، سأتابع في الوقت المناسب. أخبرني كيف حالك مع هذا التحسن.
reubenyap
19 أغسطس 2024، 12:42م
15
لا تزال قائمة الناخبين قبل التوسيع وبعده مختلفة (على الرغم من أنها متسقة حتى عبر مستخدمين مختلفين).
سيقدم @ajaydono و @Anwar مزيدًا من التفاصيل.
من المتوقع أن تكون قائمة الناخبين مختلفة.
يرجى ملاحظة أن طلب السحب هذا قد تم تقديمه ويجب أن يغلق هذه المشكلة بشكل نهائي، بعد اختباره بالفعل على مثيل رافع الخطأ عبر تفرع من المكون الإضافي للاستطلاع:
main ← merefield:ranked_choice_voter_expansion_fix
opened 07:02AM - 20 Aug 24 UTC
Currently, for Ranked Choice results and voter list expansion, the code is repla… cing the voter list with the newly fetched page of voters instead of appending to the existing voter list. This is resolved in this PR and brings it in line with regular polls.
See: https://meta.discourse.org/t/ranked-choice-poll-does-not-reflect-change-of-votes-in-outcome/321227/18?u=merefield
I've made the tests more strict to cover this and confirmed test failure with existing code without this fix.
NB I'm aware code changes around this functionality have been a little numerous of late, so this time, in the interest of minimising review burden and curtailing the iteration, I made sure the changes in this PR were fully tested in Production on the very poll which identified the issue. We've successfully tested the solution in the Production instance of the bug raiser and they have signed off. (We removed the core plugin in app.yml and cloned a fork of Poll that includes these changes).
إعجابَين (2)
سنكون ممتنين لو أمكننا دمج هذا الإصلاح، قبل أن يصبح الكود قديماً.
إعجابَين (2)
Anwar
18 سبتمبر 2024، 11:27ص
21
سيكون دمج طلب السحب هذا لطيفًا، حيث ساعد مجتمعنا بشكل كبير وسيكون مفيدًا في مجتمعات Discourse الأخرى أيضًا.
إعجاب واحد (1)
