1 |
PamOffLineSW/techmodel/EventReader.cpp reset Counter when opening a new file |
2 |
PamOffLineSW/PacketUser.cpp aggiunti apici nelle queries che davano errore MySQL |
3 |
|
4 |
---- |
5 |
PamOffLineSW/PacketUser.cpp .h |
6 |
DBScript/db.sql |
7 |
|
8 |
aggiunto .root al nome del file: rootfilename. |
9 |
tolte parti relativa a salvataggio su files e su DB di badPKT |
10 |
tolto DB ROOT_TABLE_ASS ; |
11 |
messa percentage = percentuale pacchetti con errore come double e nn come int prima |
12 |
sia sul DB sia nel codice |
13 |
Sistemati Bordi nella ricerca e gestione del merging |
14 |
|
15 |
------- |
16 |
29: |
17 |
messo lock unlock tables. |
18 |
pulito il codice |
19 |
todo: |
20 |
aggiungi info BOOT number |
21 |
|
22 |
30: |
23 |
messo apri chiudi Connessione quando serve con lock tabelle. |
24 |
prima apriva connessione all'inizio e la chiudeva alla fine. |
25 |
todo: controllare. |
26 |
Messo Boot number todo: controlla se giusto algo. |
27 |
|
28 |
TODO: togli i vari cout di debug e sistema il log |
29 |
TODO: sistema Makefile. |
30 |
|
31 |
31: |
32 |
varie |
33 |
|
34 |
01 Agosto: |
35 |
corretto baco in nome tabella per ricavare timeoffset |
36 |
messa opzione per usare una sola connessione al DB |
37 |
|
38 |
Settembre: |
39 |
Corretto spezzettamento in troppi download, messo controlli su header VRL e su CRC cadre. |
40 |
Riazzerata Contatore discontinuita per ogni nuovo download, e fatto partire da 1 |
41 |
Corretto baco che si presentava in vecchio yoda con file 05510005.pam (dovuto a lunghezza pacchetto troppo piccola) |
42 |
(in PhysEndRunReader) |
43 |
|
44 |
*** |
45 |
3 Novembre: |
46 |
sistemati bugs nella parte di merging relativi a valori obt sbagliati |
47 |
Test e cose aggiustate: |
48 |
1)errore nell'assegnare obt finale in tabella merging. files usati nei test: 05373003.pam e 05364001.pam |
49 |
"scompensi in ROOT_TABLE_MERGING per cui PKT_OBT_FINAL < PKT_OBT_INIT senza che l'OBT si sia riazzerato" |
50 |
result: OK |
51 |
2)reset OBT Pamela file 07105003.pam |
52 |
"problemi nella tabella ROOT_TABLE_MERGING quando si riazzera l'OBT di PAMELA" |
53 |
result: OK |
54 |
3)pacchetti ripetuti 05042001.pam e 05043001.pam |
55 |
"Nella tabella ROOT_TABLE_MERGING ci sono molto spesso sovrapposizioni di entry per un packet_number. Questo non deve accadere in quanto gli |
56 |
intervalli in quella tabella devono essere univoci e non sovrapporsi mai. Inoltre e` sicuramente un baco perche` al contempo l'OBT non corrisponde |
57 |
..." |
58 |
result: OK |
59 |
4)sovrapposizione tra code, 05178011.pam |
60 |
"Sempre nella tabella ROOT_TABLE_MERGING ci sono dei casi rari in cui c'e` una sovrapposizione di piu` pacchetti tra entry diverse. Questo si puo` |
61 |
riprodurre girando sul solo file 05178011.pam" |
62 |
con il nuovo sw mi sembra ok, non l'ho riprodotto con il vecchio |
63 |
i tempi iniziale di uno coincide con il finale dell'altro e sappiamo che non sono precisi ma arrotondati al secondo |
64 |
per cui al bordo ci puo' essere sovrapposizione di pacchetti |
65 |
result: OK |
66 |
5)BOOT number |
67 |
result: OK |
68 |
6)spezzettamento eccessivo su file molto corrotti, 05161005.pam |
69 |
"Ci sono dei file molto corrotti per cui chewbacca crea migliaia di files e il problema NON e` risolvibile con la flag delta_counter in quanto il |
70 |
programma identifica erroneamente dei DOWNLOAD diversi e lo spezzettamento del file e` dovuto a questo. Io ho il ricordo che anche rawreader aveva |
71 |
avuto questo problema e in qualche modo era stato risolto." |
72 |
Ora ne produce solo 6. |
73 |
result: OK |
74 |
7)contatore discontinuita' nel nome |
75 |
"Vedendo tutti questi files mi sono accorto che il contatore del file di yoda ossia YYY di yoda_OOOOO_DDD_SSS_YYY.root non si azzera mai, forse |
76 |
sarebbe carino azzerarlo ad ogni download (cosmesi)." |
77 |
ora parte da 1 e non piu da zero e si resetta ogni download |
78 |
result: OK |
79 |
8) chewbacca crasha sul file 05510005.pam |
80 |
messo controllo in PhysEndRunner |
81 |
result: OK |
82 |
9) ROOT_ID_N sballato |
83 |
"in ROOT_TABLE_MERGING succede in un certo numero non trascurabile di casi che la variabile ROOT_ID_N NON si riferisce al file corretto, ossia punta |
84 |
ad un file al cui interno NON c'e` l'intervallo in pkt_number e OBT che e` indicato nella stessa linea." |
85 |
a prima vista mi sembra ok, |
86 |
i valori del record della tabella merging devono essere contenuti in quelli del record corripondente nella tabella principale. |
87 |
result: OK |
88 |
10)nomi di default cambiati in chewbacca |
89 |
result: OK |
90 |
|
91 |
MM |
92 |
|
93 |
EM 2008-12-18: |
94 |
|
95 |
PacketUser.cpp: do not delete from root_table_merging just set good flag to zero (and check that flag when querying for merging) |
96 |
changes to handle symbolic path in the DB |
97 |
|
98 |
PacketUser.h |
99 |
#include "TSystem.h" added to handle symbolic path in the DB |
100 |
|
101 |
PamOfflineSW_Main.cpp |
102 |
changed some default values (tryMerge, step_pkt_number, step_pkt_obt, is_new_route, do_vrl_check), added -dontMerge and -no-vrl options |
103 |
changes to handle symbolic path in the DB |
104 |
|
105 |
|
106 |
EM 2009-07-29 |
107 |
|
108 |
changed to be compiled with ROOT 5.24 (added a line to LinkDef.h) |
109 |
|
110 |
compilation warnings fixed |
111 |
|
112 |
changed makefiles to have a cleaner compilation. To have the standard compilation output set the env variable "verbose" (i.e. "export |
113 |
verbose=verbose") in the shell before starting the compilation |
114 |
|
115 |
some cleaning of directories and files (*.in *.am deleted) |