[vmchecker-dev] script update note

[Claudiu Gheorghe]

Am comis acum modificarile.
De asemenea, am eliminat print-urile pentru fiecare dentry parcurs,
outputul fiind prea verbose. Acum afiseaza doar path-ul notelor
updatate.

2008/9/7 Alexandru Moşoi <brtzsnr at gmail.com>:
> m-am uitat si eu peste script. lgtm. niste chestii (maj de sintaxa) mai jos
>
> from xxx import * e descurajat (inclusiv de catre guido). import xxx
> si apoi folosesti xxx.yyy
Done

>
> muta chestiile executate in script intr-o functie main() si fa
>  if __name__ == '__main__':
>   main()
>
Done

> linia 20: transforma alea in cai absolute (cu os.path.abspath()) mai
> intai si apoi verifica directorul curent :)
Done

> docstrigurile incep cu """ nu cu """" (3 ghilimele in loc de 4)
>
Sorry, a fost un typo si l-am propagat prin copiere

> get_grade_modifdata -> grade_modification_time, si fa-l sa returneze
Done
> un timestamp si nu un string (poti sa stochezi timestampuri intr-o
> tabela sql din cate stiu eu)
Aici am stat destul de mult pe ganduri si eu.
In baza de date campul e setat ca TIMESTAMP intr-adevar, insa la o
inregistare autocompletata data arata exact in formatul in care l-am
transformat eu, si nu ca numar de secunde de la epoch time. In fine, e
posibil ca la afisare clientul ala de Sqlite sa faca si el o
conversie.
Si in orice caz, adaptorul de python converteste timestampul din
sqlite in string unicode, asa ca aleg sa il las asa cum este acum, in
stringuri.

>
> 142: functia poti sa o transformi in (e mai lizibil)
> with open(grade_filename) as f:
>  value = int(f.read())
True. Am citit acum cate ceva despre instructiunea `with`. Nu e chiar
banal ce se intampla in spatele ei, sau poate ca sunt prea obosit acum
ca sa inteleg

>
>
> 195,196: fara ; la sfarsit de linie
da, sorry

>
> 84: un singur return id_student e suficient :)
penibil, intr-adevar :)

> 104: nu e nevoie de paranteze pentru tuple-ul ala
>
Aici n-am inteles de ce sunt parantezele de prisos

>
> 2008/9/7 Ana Savu <ana.savu86 at gmail.com>:
>>
>>
>> 2008/9/7 Claudiu Gheorghe <claudiugh at gmail.com>
>>>
>>> Am comis scriptul de update al notelor.
>>> Astept un cod review.
>>>
>>> --
>>> Gheorghe Claudiu-Dan
>>> Student la Facultatea de Automatica si Calculatoare, UPB
>>> tel: +40 726 758 198
>>
>> De ce nu faci scriptul executabil?
Eu le fac pe windows si nu vreau sa va bag C:\pytgon25\python.exe.
In fine, am pus #! /usr/bin/pyhon
>>
>> Incearca sa folosesti modulul 'misc' pentru a afla VMCHECKER_ROOT pentru
>> uniformitate
>>      17: vmchck_root  = misc.vmchecker_root()
Done

>>
>> In rest e fi bine. Good work!
>>
>> --
>> Ana
>>
>> _______________________________________________
>> vmchecker-dev mailing list
>> vmchecker-dev at lists.rosedu.org
>> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev
>>
>>
>
>
>
> --
> Alexandru Moşoi
> http://alexandru.mosoi.googlepages.com
>
> Please avoid sending me Word or PowerPoint attachments.
> See http://www.gnu.org/philosophy/no-word-attachments.html
> _______________________________________________
> vmchecker-dev mailing list
> vmchecker-dev at lists.rosedu.org
> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev
>



-- 
Gheorghe Claudiu-Dan
Student la Facultatea de Automatica si Calculatoare, UPB
tel: +40 726 758 198