Skip to content
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

Merge the firmware variants to streamline codebase #11

Closed
kaklik opened this issue Feb 17, 2023 · 1 comment
Closed

Merge the firmware variants to streamline codebase #11

kaklik opened this issue Feb 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@kaklik
Copy link
Contributor

kaklik commented Feb 17, 2023

Současný firmware má několik různých variant, které se jen minimálně liší:

  • Počtem kanálů 256/512
  • Jestli má nebo nemá SDkartu pro zápis
  • Frekvencí použitého krystalu

U těchto rozdílů zřejmě neexistuje důvod proč držet různé codebase. Zádrhelem je akorát to, že by bylo potřeba implementovat přepínací vlastnosti firmware (což je ale stejně potřeba kvůli integračnímu intervalu). Tyto parametry jsou buď nastaveny při překladu (frekvence krystalu), nebo jsou parametrem přístroje (počet kanálů, integrační doba, zápis na SDkartu...)

Parametry přístroje by zřejmě správně měly být zapisovány do externí EEPROM, aby nebyly přepsány aktualizací firmware.


Poznámka k počtu kanálů
Původní verze detektoru měla pouze 256 kanálů jako liulin. Zvětšení počtu kanálů podle @kakl vedlo na problem, ze ta ATmega ma nejakou proprietarni kompenzaci DC offsetu. Díky tomu interní ADC nefunguje dobře s externí referencí a způsobuje to jitter kanálů na spektrogramu. #15

Počet kanálů by ale obecně potřeboval zvýšit viz UniversalScientificTechnologies/GEODOS01#23 Tudíž je zřejmě nutné začít používat externí ADC.

@kaklik kaklik added the enhancement New feature or request label Feb 17, 2023
@kaklik
Copy link
Contributor Author

kaklik commented Apr 21, 2023

Tohle bylo provedeno až na firmware pro "UAVDOS", který zde zatím zůstal.

@kaklik kaklik closed this as completed Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant