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

Zlepšit umístění TFRPM pcb #76

Open
1 task
roman-dvorak opened this issue Nov 24, 2020 · 7 comments
Open
1 task

Zlepšit umístění TFRPM pcb #76

roman-dvorak opened this issue Nov 24, 2020 · 7 comments
Labels
CAD work Design development and edits Industrial Improvements enabling serial production

Comments

@roman-dvorak
Copy link
Member

roman-dvorak commented Nov 24, 2020

Bylo by dobré, do poličky uprosřed pilonu vylepšit připevňení senzoru tak, aby matka v něm byla zapuštěná (tj. aby nebyl potřeba klíček a umístit ho více do středu tak, aby hrana PCB nekolidovala s bočnicemi pylonu.

Zároveň by nebyla špatná nějaká krabička, která by to zakrytovala před vodou. Zároveň je potřeba vyřešit nějaké svedení všech kabelů z rotorové hlavy.

  • zlepšit viditelnost TFRPM indikační LED. Aktuálně je na poličce vidět na denním světle akorát zespodu a to ještě mizerně.
@roman-dvorak roman-dvorak changed the title Zlepšit umístění RPM PCB Zlepšit umístění TFRPM PCB Nov 24, 2020
@roman-dvorak roman-dvorak changed the title Zlepšit umístění TFRPM PCB Zlepšit umístění TFRPM pcb Nov 24, 2020
@kaklik
Copy link
Member

kaklik commented Nov 25, 2020

Asi bych potřeboval nějaký obrázek, abych pochopil, co tohle issue znamená.

@roman-dvorak roman-dvorak added the Industrial Improvements enabling serial production label Nov 29, 2020
@roman-dvorak
Copy link
Member Author

Původně to bylo myšleno tak, že se přesune senzor do prostoru střední příčky pylonu, kde se na něj vytvoří nějaký "držák".

Nyní bych navrhoval TFRPM dát někam do spodní části vírníku (někam do prostoru ESC a autopilota). Pro začátek asi v TFRPM01 v krabičče, ve které je dodávaný.

Tohle z důvodu, že RPM kablík se snadno vytrhne z rotorové hlavy (z TFPROBE) a nebude se tahat za I2C vedení při možné nehodě.

@roman-dvorak roman-dvorak self-assigned this Dec 2, 2020
@kaklik kaklik added the CAD work Design development and edits label Dec 15, 2020
@kaklik
Copy link
Member

kaklik commented Feb 17, 2021

A jak to teď vlastně je s použitím TFRPM driveru na stejné sběrnici jako TFSLOT? Funguje to, nebo ne?

Protože k TFSLOT se stejně I2C na pylon bude tahat, takže nevidím moc rozdíl v tom tam vést dvakrát I2C, nebo I2C a servokablík.

Z hlediska HW by bylo nejlepší řešení jen jedno I2C na které by byly připojeny oba senzory, díky tomu, že TFRPM je průchozí. Vím ale že dříve jsme s tímto řešením měli potíže. Je tak otázka zda je to však relevantní i pro současný software?

@slimonslimon
Copy link
Contributor

Tady asi chybí nějaké "měření/pokusy" který by zjisti, kde je problém.

Driver neprovádí novou inicializaci RPM senzoru. Ale je potřeba vyzkoušet že reset rpm driveru stačí na obnovení funkčnosti.

@roman-dvorak
Copy link
Member Author

Na vírníku 4S tohle bude TFRPM integroaný do blízkosti TFSLOT. Proto je toto issue již nerelevantní.

@kaklik
Copy link
Member

kaklik commented Mar 22, 2021

Issue lze zavřít, až když bude reálně vyřešené. K tomu je potřeba upravit víčko TFSLOT #16 tak, aby tam bylo možné tam chytit i TFRPM01 PCB.

@kaklik kaklik reopened this Mar 22, 2021
@kaklik
Copy link
Member

kaklik commented Apr 2, 2021

Tady asi chybí nějaké "měření/pokusy" který by zjisti, kde je problém.

Driver neprovádí novou inicializaci RPM senzoru. Ale je potřeba vyzkoušet že reset rpm driveru stačí na obnovení funkčnosti.

Z tohoto experimentu vyplynulo, že snímače na jedné sběrnici fungují a čtení nepadá. Co ale nefunguje je měření otáček, které při takovém zapojení občas dává zápornou hodnotu otáček..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CAD work Design development and edits Industrial Improvements enabling serial production
Projects
None yet
Development

No branches or pull requests

3 participants