ATNEL tech-forum

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-11T11:43:03+01:00

sq8dsr napisał(a):

A jak muszę wspomnieć o dodatkowym efekcie który objawił się jako "skutek uboczny" biblioteki

No proszę .... warto bo to też ważna technika programowania jak widać

Statystyki: Napisane przez mirekk36 — 11 mar 2017, o 11:43

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-11T11:23:23+01:00

A jak muszę wspomnieć o dodatkowym efekcie który objawił się jako "skutek uboczny" biblioteki
Mianowicie przekonałem się w końcu do stosowania warunków preprocesora i kompilacji warunkowej.
Przerabiam obecnie bibliotekę do możliwość obsługi dwóch enkoderów
Założenie jest takie, że ma obsługiwać dwa enkodery, ale nawet żeby to mogły być enkodery różne np. half i full.

Przepisywanie z poradnika ma jednak spore zalety hehehe

Statystyki: Napisane przez sq8dsr — 11 mar 2017, o 11:23

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-07T08:44:18+01:00

matw napisał(a):

Druga sprawa, to sprawdź tą linijkę w encoder_proc(), bo jakby brakuje ci nawiasów i wg. mnie to może być kluczowe:

hahaha, no i nie ma to jak świerze oko, głupi błąd. Dzięki kolego, wielkie.
Oczywiście to była kluczowa sprawa. Widać zbyt długo gapiłem się w ten kod i się już znieczuliłem na błędy
Oczywiście teraz działa wszystko bardzo dobrze. Zarówno na enkoderze pełno jak i półkrokowym.
Teraz muszę rozbudować bibliotekę, żeby można była zdefiniować i obsłużyć jednocześnie dwa enkodery.

Statystyki: Napisane przez sq8dsr — 7 mar 2017, o 08:44

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-06T18:53:12+01:00

matw napisał(a):

bo jakby brakuje ci nawiasów i wg. mnie to może być kluczowe:

i to jest kluczowe bo bez nawiasów to leci sieczkarnia jeśli chodzi o wyrażeni warunkowe ?: .... jedno łączy się z drugim w nieodpowiednim miejscu

ale wartości w tablicach to też klucz

No! panie matw ! .... otrzymujesz tytuł SOKOLE OKO

Statystyki: Napisane przez mirekk36 — 6 mar 2017, o 18:53

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-06T18:17:45+01:00

@sq8dsr, przyjrzałem się wstawce którą zamiesiłeś i porównałem ze swoim kodem.
Ja mam tylko enkoder pełnokrokowy i z nim biblioteka napisana z poradników Mirka działa wyśmienicie.

Znalazłem dwie rozbieżności, które mogą mieć wpływ na twoje problemy.
Tablica dla połkrokowego wygląda ok, natomiast dla pełnokrokowego jest mała różnica w stosunku do mnie, jedna wartość u Ciebie jest 0x05, a ja mam 0x00, mój kod:
[syntax=c]const uint8_t encoder_tab[7][4] PROGMEM = {
{0x00, 0x02, 0x04, 0x00}, {0x03, 0x00, 0x01, 0x10},
{0x03, 0x02, 0x00, 0x00}, {0x03, 0x02, 0x01, 0x00},
{0x06, 0x00, 0x04, 0x00}, {0x06, 0x05, 0x00, 0x20},
{0x06, 0x05, 0x04, 0x00}
};[/syntax]
Druga sprawa, to sprawdź tą linijkę w encoder_proc(), bo jakby brakuje ci nawiasów i wg. mnie to może być kluczowe:
[syntax=c]register uint8_t ABstate = ( (pin & ENCODER_B)? 2 : 0 ) | ( (pin & ENCODER_A)? 1 : 0 );[/syntax]

Statystyki: Napisane przez matw — 6 mar 2017, o 18:17

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-06T09:07:47+01:00

W celach diagnostyki wyświetlam na lcd wartości, które pobierane są z tablicy enc_tab. Czyli wartość zmiennej enc_stat (z funkcji encoder_proc)
Nie wiem czemu ale nie otrzymuję wartości typu 0x13, 0x20, 0x23, generalnie tych wartości które "sięgają" do starszej połówki bajtu i po operacji:
ABstate = (enc_stat & 0x30);
dają wartości niezerowe (np. 0x10).

oto moje tablice:
[syntax=c]//encoder półkrokowy
const uint8_t enc_tab[6][4] PROGMEM = {
{0x3, 0x2, 0x1, 0x0}, {0x23, 0x0, 0x1, 0x0},
{0x13, 0x2, 0x0, 0x0}, {0x3, 0x5, 0x4, 0x0},
{0x3, 0x3, 0x4, 0x10}, {0x3, 0x5, 0x3, 0x20}
};[/syntax]

[syntax=c]//encoder pełnokrokowy
const uint8_t enc_tab[7][4] PROGMEM = {
{0x0, 0x2, 0x4, 0x0}, {0x3, 0x0, 0x1, 0x10},
{0x3, 0x2, 0x0, 0x0}, {0x3, 0x2, 0x1, 0x0},
{0x6, 0x5, 0x4, 0x0}, {0x6, 0x5, 0x0, 0x20},
{0x6, 0x5, 0x4, 0x0}
};[/syntax]

i funkcja obsługująca tablicowane przeliczanie kroków:
[syntax=c]void enkoder_proc (void) {
static uint8_t enc_stat;
register uint8_t pin = ENC_AB_PIN;
register uint8_t ABstate = ( (pin&ENC_B)?2:0 | (pin&ENC_A)?1:0 );
enc_stat = pgm_read_byte( &enc_tab[enc_stat & 0x0f][ABstate] );
ABstate = ( enc_stat & 0x30 ); //po tej operacji wartości ABstate są ciągle równe zero więc późniejszy if się nie wykonuje.
if ( ABstate ){
enco_dir = ABstate;
if (ABstate == ENC_RIGHT) enco_cnt++;
else enco_cnt--;
enco_flag = 1;
}
}[/syntax]

Statystyki: Napisane przez sq8dsr — 6 mar 2017, o 09:07

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-06T07:33:15+01:00

To ja mogę tylko napisać, że także jestem po przepisaniu biblioteki z poradników i działa po prostu miodnie
Ja mam encoder pełnokrokowy.

Statystyki: Napisane przez MikieMan — 6 mar 2017, o 07:33

Re: Enkoder od podstaw ? - warto ? ;)

2017-03-05T23:05:39+01:00

Cześć

Staram się właśnie od kilku godzin ogarnąć bibliotekę Mirka do enkoderów.
Mam całą przepisaną. Co więcej nawet dokonałem jej analizy
Tzn chodzi mi o mechanizm tablicowania (callbacki chyba się po warsztatach dobrze wryły).
No i mam zagwozdkę.
Testuję dwa enkodery, na zestawie ATB:
Na razie na INT0 i INT1
pełnokorokowy:
11 - tryk
01
00
10
11 - tryk

i chyba półkrokowy:
11 - tryk
01
00 - tryk
10
11 - tryk

Bibliotekę przepisałem dokładnie, zresztą analizując ją, nie widzę błędów, wszystko wydaje się być spójne.
ale jest problem:

Otóż na półkrokowym zdarzenie ENCODER_EVENT, przy pierwszym pokręceniu (po resecie) wykonuje się jeden raz następnie przestaje reagować.
Przerwania działają.
Funkcja encoder_proc nie chce już poraz kolejny ustawić flagi enco_flag.
Analizuję jakie wartości są pobierane z tablicy enc_tab przy kolejnych stanach ABstate (pinów encodera) i wydaje się, że powinno być ok.
Wszystko się kompiluje bez błędu.

Pełnokrokowy nie reaguje w ogóle (oczywiście przestawiam definicję w pliku .h).

Na razie nie wstawiam kodu, może ktoś miał podobnie, to musi być jakiś głupi błąd....

Statystyki: Napisane przez sq8dsr — 5 mar 2017, o 23:05

Re: Enkoder od podstaw ? - warto ? ;)

2016-12-22T08:07:08+01:00

damian29 napisał(a):

Rozpisałem enc_tab na kartce i przeanalizowałem na spokojnie

Zawsze to polecam w wielu poradnikach ... to musi pomóc

Statystyki: Napisane przez mirekk36 — 22 gru 2016, o 08:07

Re: Enkoder od podstaw ? - warto ? ;)

2016-12-21T22:49:57+01:00

I własnie już do tego doszedłem. Rozpisałem enc_tab na kartce i przeanalizowałem na spokojnie jak te indeksy się zmieniają w zaleznosci od ABstat i jest tak jak mówisz.

Enkoder już liczy

Dzięki za zainteresowanie w sumie błahym problemem.

Pozdrawiam i Wesołych Swiąt

Statystyki: Napisane przez damian29 — 21 gru 2016, o 22:49

Re: Enkoder od podstaw ? - warto ? ;)

2016-12-20T20:11:56+01:00

A co to jest ABstate w drugim indeksie?
Natomiast pierwszy indeks to 4 młodsze bity zmiennej enc_stat które się przecież zmienia bo po każdym "tyknięciu" przypisujesz mu właśnie nową wartość w zależności od starej wartości i ABstate

Statystyki: Napisane przez MirkoT — 20 gru 2016, o 20:11

Re: Enkoder od podstaw ? - warto ? ;)

2016-12-20T17:36:02+01:00

Witam

Mam pytanie dotyczące funkcji void enkoder_proc () , a mianowicie nie rozumie w jaki sposób do zmiennej enc_stat kopiowane są wartosci z tablicy enc_tab. Chodzi mi głównie o pierwszy nawias kwadratowy enc_stat=( pgm_read_byte&[enc_stat & 0x0f][ABstate]). Jak to się dzieje że w zmiennej enc_stat może pojawić się np. 1 , 2 , 3 itd ?? Nie widzę w tej funkcji np. enc_stat++ żeby ją zwiększyć zmienna chociaż o jeden żeby mięć dostęp do następnego wiersza w tablicy enc_tab.

Proszę o pomoc w tym zakresie pozdrawiam

Chciałem jeszcze dodać że jestem nowym użytkownikiem tego forum i jest to mój pierwszy post.

Statystyki: Napisane przez damian29 — 20 gru 2016, o 17:36

Re: Enkoder od podstaw ? - warto ? ;)

2016-11-25T17:49:27+01:00

ja z chęcią to sprawdzę ale na razie jestem daleko od tematu enkoderów - nie mnie jednak - NA PEWNO niedługo powrócę bo będzie mi potrzebna obsługa enkodera do jednego z kolejnych poradników

Statystyki: Napisane przez mirekk36 — 25 lis 2016, o 17:49

Re: Enkoder od podstaw ? - warto ? ;)

2016-11-25T17:46:32+01:00

Witam
Mam komentarz do cz. 5 poradnika o enkoderach. https://youtu.be/YnYDnZWSnl8?t=4m3s

Chciałbym zauważyć, że gdy w enkoderze określanym tu jako pełnokrokowy o przebiegu:

00
10
11
01
00 TRYK

zamienimy miejscami pin wspólny (GND) z pinem A lub pinem B to powinniśmy otrzymać przebieg:

00
10
11
00
00 TRYK

Taki oto właśnie dziwny przebieg ma jeden z pokazywanych enkoderów. Coś mi się wydaje że ten rzekomo dziwny enkoder, ma po prostu inaczej rozmieszczone wyprowadzenia (wspólny pin nie znajduje się na środku). I po zamianie miejscami odpowiednich wyprowadzeń, enkoder stanie się normalny . To są tylko moje przypuszczenia. Pewnie Pan, Panie Mirku podczas testów wykluczył taką przyczynę dziwności tego enkodera? Ale może umknęło to uwadze.

Dodam jeszcze raz że, gdyby ktoś zamienił wyprowadzenia to uzyska taki sam efekt.

PS
Nie mam wiele praktycznego doświadczenia z enkoderami, więc jeżeli się mylę to proszę mnie poprawić. A o tych nietypowych kodowaniach słyszę pierwszy raz.

Statystyki: Napisane przez wojtekr51 — 25 lis 2016, o 17:46

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T16:48:50+01:00

no bo przekombinowałeś po pierwsze to najpierw ustal o jaki zakres tobie chodzi ? od zera do ilu ? do 65535 ? czy mniej czy więcej ?

Bo jeśli od 0 do 65535 to przede wszystkim musisz sobie zmienić typ zmiennej, która przechowuje licznik enkodera na int32_t ! Wtedy będziesz mógł go zerować we własnej funkcji tak jak chciałeś wyżej gdzie pisałem, że idziesz dobrą drogą

I nie będziesz potrzebował, tak jak mówiłem, robić zmian w bibliotece do enkodera, bo o te mi chodziło - a oczywiście jeśli chodzi o wyświetlanie na LCD liczb dodatnich większych niż 32768 to chyba normalne że musisz sobie dodać w bibliotece LCD nową funkcję (ale to jest niejako zadanie do książki dla każdego czytelnika)

Statystyki: Napisane przez mirekk36 — 11 kwi 2016, o 16:48

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T16:35:46+01:00

Niestety ani moje wypociny, ani kod kolegi Sparrow-hawk nie ograniczają wyświetlania do liczb dodatnich.

Quote:

nie kombinuj w ciele bibliotek jeśli nie rozumiesz jak one dokładnie działają, po co ? po co ?

Dopiero ingerencja w Lcd_int ze zmianą atoa() na ltoa() wyświetla tylko liczby dodatnie ( oczywiście ograniczam wyświetlenie 65535 przy kręceniu w lewo ).

Dzięki wszystkim za pomoc.

Statystyki: Napisane przez acid255 — 11 kwi 2016, o 16:35

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T09:05:39+01:00

Sparrow-hawk napisał(a):

Dziwi mnie, że kompilacja tego kodu nie generuje żadnych ostrzeżeń. Przecież ten warunek:

Kolega wyżej napisał, że dopiero to przetestuje - i może warto było dać mu szansę to przetestować we własnym zakresie? - dlatego ja nie pisałem o tych szczegółach. Bo często NIC tak nie nauczy kogoś jak doświadczenie na własnej skórze o co chodzi z typami .... Tymczasem to jest właśnie śliczny przykład - na którym nagle by wyszło o co chodzi

Statystyki: Napisane przez mirekk36 — 11 kwi 2016, o 09:05

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T08:08:48+01:00

[syntax=c]uint_16t licznik; // nowa zmienna globalna dla zliczania impulsów dodatnich

void Moja_funkcja ( void ) {
licznik = get_encoder();
if ( licznik<0 ){
licznik=0; // licznik impulsów z enkodera
set_encoder( 0 ); // to daję po to aby zmienna enco_cnt nie była ujemna
}
}[/syntax]
Dziwi mnie, że kompilacja tego kodu nie generuje żadnych ostrzeżeń. Przecież ten warunek:
[syntax=c]if ( licznik<0 ) {
...
}[/syntax] nigdy nie będzie spełniony. Mało tego, dla ujemnych wartości zwracanych przez get_encoder(); rezultaty działania będą odmienne od oczekiwanych.

Więc albo zapiszemy:
[syntax=c]int16_t licznik;[/syntax]
Albo zapiszemy to tak:[syntax=c]uint16_t counter;

void my_function ( void ) {
int16_t value;
if((value = get_encoder()) < 0)
counter = (value + 65535) + 1;
else
counter = value;
}[/syntax]

PS. Nie należy mieszać nazw polskich z anglojęzycznymi.

Statystyki: Napisane przez Sparrow-hawk — 11 kwi 2016, o 08:08

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T07:00:54+01:00

No właśnie jeszcze nie, bo akurat jestem w pracy - dziś to przetestuję.
Dziękuję za pomoc.

Statystyki: Napisane przez acid255 — 11 kwi 2016, o 07:00

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T06:48:49+01:00

Dokładniuśko tak ... no ale chyba sam widzisz, że tobie teraz to działa - tak ? czy nie ?

Statystyki: Napisane przez mirekk36 — 11 kwi 2016, o 06:48

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-11T06:34:20+01:00

czyli w mojej funkcji musi być taki ogranicznik wraz z nową zmienną dla zliczania impulsów:

[syntax=c]uint_16t licznik; // nowa zmienna globalna dla zliczania impulsów dodatnich

void Moja_funkcja ( void ) {
licznik = get_encoder();
if ( licznik<0 ){
licznik=0; // licznik impulsów z enkodera
set_encoder( 0 ); // to daję po to aby zmienna enco_cnt nie była ujemna
}
}[/syntax]

?

Statystyki: Napisane przez acid255 — 11 kwi 2016, o 06:34

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-10T22:34:36+01:00

acid255 napisał(a):

ale nie wiedzieć czemu to nie działa.

A tyle razy też powtarzam - nie kombinuj w ciele bibliotek jeśli nie rozumiesz jak one dokładnie działają, po co ? po co ?

Przecież masz wołanego callbacka, twoją własną funkcję w której odczytujesz wartość enkodera i to w sposób atomowy - TAM ograniczaj czyli już u siebie w swoim kodzie - a nie w bibliotece

Statystyki: Napisane przez mirekk36 — 10 kwi 2016, o 22:34

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-10T20:34:14+01:00

Źle wstawiony kod.

Statystyki: Napisane przez Jaglarz — 10 kwi 2016, o 20:34

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-10T20:49:30+01:00

Właśnie moje próby z ograniczeniem zliczania tylko do 0 zacząłem w bibliotece enkodera w funkcji encoder_proc
[syntax=c]if( ABstate == ENC_RIGHT ) enco_cnt++;
else enco_cnt--;
enco_flag = 1;[/syntax]
zmieniłem na:
[syntax=c]if( ABstate == ENC_RIGHT ) enco_cnt++;
else if (enco_cnt >= 0 ) enco_cnt--;
enco_flag = 1;[/syntax]

ale nie wiedzieć czemu to nie działa.

Do funkcji Lcd_int() też zaglądałem ale nie wpadłem by zaglądnąć do BB a tam na tłusto jak byk na str. 232 wszystko jest, tak więc będę próbował zrobić sobie Lcd_long z wykorzystaniem ltoa().
Tak na marginesie: tyle razy oglądałem poradniki na których tyle razy było powtarzane o wykonaniu sobie odpowiednich funkcji dotyczących konwersji i byłem pewien, że takiego prymitywnego błędu nie zrobię, a jednak.

Statystyki: Napisane przez acid255 — 10 kwi 2016, o 19:55

Re: Enkoder od podstaw ? - warto ? ;)

2016-04-09T23:33:53+01:00

Daro69 napisał(a):

tak sobie myślę czemu nie tak?
bez przerabiania bibliotek.

Od tego powinno się zacząć a nie od próby wymuszania zera przez uint16_t. Przecież przy zmniejszaniu - gdy dojdzie do zera pojawi się wtedy 65535 nawet jeśli zmieni się w bibliotece i wcale nie będzie to ładnie

Więc takim warunkiem - ogranicznikiem jak najbardziej warto pomajstrować.

Statystyki: Napisane przez mirekk36 — 9 kwi 2016, o 23:33