UniversalScientificTechnologies / AIRDOSC01

Airborne Scintillation based ionizing radiation detector.
https://www.ust.cz/UST-dosimeters/
GNU General Public License v3.0
7 stars 2 forks source link

Dokumentace #4

Open kaklik opened 2 years ago

kaklik commented 2 years ago

Modul SiPM má v této konstrukci již několik variant. Jak je vidět ve složce hw.

Momentálně implementovaný způsob zacházení s verzemi ale vede k tomu, že není jasné co je aktuální verze. Například v konstrukci GEODOSu se používá PCB AIRDOSC01A_PCB01B. Přitom by zřejmě bylo logičtější zachovat původní název a vytvořit například modul SIPM02A. Uvnitř modulů MLAB. Důsledky aktuálního stavu jsou:

Navrhoval bych tento modul od dokumentace AIRDOSC odtrhnout do nového repozitáře v mlab-modules. Zároveň tento modul přejmenovat společně s přechodem na jeho novější verzi. Moduly které se dnes zřejmě nepoužívají, zřejmě například SiPM01B by měly být taktéž v samostatném repozitáři v mlab modules.

Konvenci pojmenování modulů jako AIRDOSC01A_PCB01B bych v budoucnu navrhoval používat pouze pro PCB, které s jistotou nelze samostatně používat, nebo použít v jiném zařízení a jsou tudíž spolehlivě MLAB nekompatibilní.

kaklik commented 2 years ago

Pokusil jsem se to vyřešit vytvořením samostatného repozitáře SIPM v mlab-modules. Jenže se při přesunu hnad prvního modulu SIPM01A ukázalo, že zároveň existuje i v repozitáři Modules. Přesunutí tak není jednoduché, neboť oba moduly se stejným názvem mají rozdílnou historii změn

kaklik commented 2 years ago

Tak jsem částečně uklidil vytvořením nových MLAB repositářů s moduly SIPM02 a SIPMPOWER01. Nedovedu ale vyřešit problém s dokumentací SIPM01.

ChroustJan commented 2 years ago

SIPM01 není možné sloučit. Jsou to dva rozdílné moduly. Ten v MLAB repozitáři nemá výrobní data. Očekával bych tedy, že se ani nevyráběl. Proto bych navrhoval, pokud chceme SIPM01 z UST přesunout do MLAB modulů ho přejmenovat na SIPM03. A u UST SIPM01 zachovat jeho název. SIPM01 v UST AIRDOSC01 má ještě verzi SIPM01B.

kaklik commented 2 years ago

SIPM01 není možné sloučit. Jsou to dva rozdílné moduly. Ten v MLAB repozitáři nemá výrobní data. Očekával bych tedy, že se ani nevyráběl. Proto bych navrhoval, pokud chceme SIPM01 z UST přesunout do MLAB modulů ho přejmenovat na SIPM03. A u UST SIPM01 zachovat jeho název. SIPM01 v UST AIRDOSC01 má ještě verzi SIPM01B.

Tuhle úvahu nechápu. Pokud SIPM01 v repozitáři AIRDOSC nahrahuje ten modul, který je v MLAB modules o kterém navíc tvrdíš, že se nikdy nevyráběl. Tak nevidím žádný důvod ho přejmenovávat. Protože název SIPM01A i SIPM01B je kompatibilní s MLAB a taktéž návrh toho modulu je MLAB kompatibilní. Tudíž to vidím tak, že zdejší modul v AIRDOSC nahrazuje ten původní v MLAB, který se asi nějakým omylem stal balastem v repozitáři.

ChroustJan commented 2 years ago

Navrhuji ten MLAB balast přejmenovat na SIPM03. Přijde mi škoda ten návrh zahodit. Stál nějaký čas a nevím zda se někdy nehodí. Byla to iterace.

kaklik commented 2 years ago

image