TIS2019-FMFI / ims-spectrometer

Software for measurements with IMS spectrometer
0 stars 0 forks source link

Návrh celok #52

Closed krivanek06 closed 4 years ago

krivanek06 commented 5 years ago

Tu budú popísané problémy, čo treba zmeniť na návrhu pred tým ako to pošleme:

krivanek06 commented 5 years ago

zmeny:

--Skús aj parafrázovať vety ktore som napísal a ktoré sú v docs, nech sa neopakujú slová, --Niekde sa ešte vyskytuje slovo "dáta" aj "údaje". Nahrať to len jedným z nich (viem, aj ja som písal jedno aj druhé) , ale použime len jedno slovo.


Ak toto všetko je, tak to submitneme

krivanek06 commented 5 years ago

Okej, spravili sme teda návrh verziu 1 , diki chlapci za spoluprácu. Prihadzujem link : https://github.com/TIS2019-FMFI/ims-spectrometer/blob/master/application/Documents/TIS%20-%20n%C3%A1vrh.pdf

teraz čakáme na review.

pavelpetrovic commented 4 years ago

moje poznamky z dnesneho stretnutia:

Juraj a Eduard nepritomni

za posledne dva tyzdne chybaju zapisnice chybaju aj dve starsie zapisnice zapisnice sa pisu vzdy priamo na stretnuti a na konci stretnutia sa stlaci [SAVE]. potom nemozeme byt v stave ako sme teraz.

nevieme, ci sa shieldbuddy bude menit kod, alebo nie chyba mi v navrhu komunikacny protokol medzi shield buddy a PC aplikaciou a pripadne aj rozhranie Shild Buddy s pristrojom, ak sa v nom meni kod (alebo medzi novym a starym kodom...) sekcia kom. protokoly obsahuje zbytocne encyklopedicke informacie, ale neobsahuje konkretnu strukturu a obsah kom. protokolu rovnako formaty suborov su rozpisane podrobne ale v sekcii CSV format, prosim adresne prehodit pod jednotlive polozky v 3.1.1 ale chyba nam popis formatu suboru s nastaveniami potrebujem vediet, ci sa tim stretol so zadavatelom s navrhom UI a podrobne presli celu funkcionalitu na obrazkoch na zaciatku navrhu UI sa prosim odkazat na obrazok UI v KP (ak stale plati v takej verzii) prosim doplnit aspon kratky vysvetlujuci text ku kazdemu diagramu (2 odstavce?) prosim vytvorit novy stavovy diagram pre konkretnu entitu a jej pasivne stavy, medzi ktorymi sa prechadza stavovymi prechodmi trigerovanymi akciou, udalostou alebo casovym intervalom navrh po schvaleni netreba menit, ale vo vyslednej technickej dokumentacii je potrebne aby to sedelo s vyslednou implementaciou, napr. si vytvorte jeden issue, v ktorom si zaznacite vsetky nove zmeny oproti navrhu pocas implementacie testovacie scenare ako dokument prosim vypracovat, testy v kode su dolezite a chvalim velmi dolezite: treba rovnomerne rozdelit pracu na implementacii, aj za cenu neoptimalneho vysledku. dolezitejsie je aby sa kazdy rovnomerne zapojil ako ze vysledok bude dokonaly.

napiste mailom, ked bude nova verzia navrhu a tiez testovacie scenare. v tyzdni od 9.12. skuste co najviac dokoncit implementaciu a v poslednom tyzdni sa urcite stretnime (ak nemozete v piatok tak v inom case)

krivanek06 commented 4 years ago

Návrh sa zmenil podľa požiadaviek, link : https://github.com/TIS2019-FMFI/ims-spectrometer/blob/master/application/Documents/TIS%20-%20n%C3%A1vrh.pdf

krivanek06 commented 4 years ago

Navrh je teda ok , zrkadlovy graf po dohode s pánom Matejčíkom nebude.

pavelpetrovic commented 4 years ago

A kto je na vine, ze zrkadlovy graf nebude?

krivanek06 commented 4 years ago

Z intenzitného grafu na základe zafarbernie bodov sa dá vyčítať krivka grafu z ktorého vznikla., preto zrkadlový graf je nepotrebný. Radšej sme intenzitný graf natiahli na celú dĺžku obrazovky kvôli prehľadnosti. O povolení vykovaní týchto zmien mám emailové potvrdenie od pána Matejčíka.

pavelpetrovic commented 4 years ago

viem si predstavit, ze ste prof. Matejcika ukecali, ale mne sa nepaci takyto ustup. z farieb sice nieco vidno, ale je predsa ine, ak to vidim v priebehu na x-y grafe, nie? ved preto to tam povodne malo byt. suhlasim s tym, ze ak tam je toho privela naraz, tak to moze byt neprehladne, ale znamena to, ze spatne uz nie je mozne taky graf zrekonstruovat? nie je to skoda?