-
Notifications
You must be signed in to change notification settings - Fork 8
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
syöte h rikkoo interpoloinnin #16
Comments
Ilmeisesti oleinen vain tapauksissa, joissa 2 osatehtävää tyylillä a+b*60, jossa b sakkojen määrä. Kun tässä jonkun vartion suoritus on h (a=h, b=h), menee interpolointi pilalle koko sarjan osalta. Laitan vielä xml-tiedoston, jossa asia havainnollistettu. |
Tein testikisan, jossa sarjoina e- ja h-syötteitä eri yhdistelminä, ja tehtävänä aikaväli, niin pienin kuin suurin -interpoloinnilla. |
Teinpä sitten kokeita asiasta ja lopputuloksena committi, jossa e on lisätty interpoloinnista ulkopuoliseksi vievien listaan, sekä h- syötteelle tehty looginen negaatio koodiin. Ja nyt näyttää ainakin minulla toimivan. Omassa forkissani on paljon muitakin muutoksia, niin katsotaan saanko tehtyä tänne sopivan pull requestin jollain tapaa... |
En tiedä, onko tuo korjaus toivottu. Esimerkkitehtävä: max pisteet 4, maksimisuoritusaika 10 min aika jolloin rastihlö keskeyttää suorituksen:
(Toinen konkreettinen esimerkki, mihin tarvitaan osatehtäväkohtaista hylkäämistä, on nyt syys-SM 2021 tehtävä Labyrintti.) Nyt tuon voi tehdä kipaan siten, että on kaksi osatehtävää: nopein aika (a) ja oikeat vastaukset (b). Jos vartio ylittää maksimiajan, se saa osatehtävästä a hylätyn (h), mutta osatehtävästä b edelleen pisteet. Vartion suoritusajaksi ei voi merkitä esim. 10 minuuttia, koska silloin mediaani määritetään väärin. Nyt ehdotettu korjaus siis ilmeisesti estää tämän toteutuksen. |
Joissakin tapauksissa yksittäisen vartion saama h (hylätty) pilaa interpoloinnin muiden vartioiden osalta. Vertailin kokonaislukuja 1,5x interpoloinnilla. Kun yksittäinen vartio sai h, sai paras vartio 2,7/5.
The text was updated successfully, but these errors were encountered: