ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov
5
0
Fork 0
Code Issues 2 Pull Requests Projects Releases Wiki Activity

Энкодер. Вопросы 01 #3

New Issue
Open
opened 2023-11-08 08:25:36 +00:00 by stud126186 · 89 comments
stud126186 commented 2023-11-08 08:25:36 +00:00
Member
Copy Link

Encoder.c Line 14 in 073d5e43f6
ISR(TIMER1_COMPA_vect) {

Вынести из прерывания.

Encoder.c Line 15 in 073d5e43f6
next_state = PIND & PIND_MASK;

Значение next_state должны быть одним из 0x01, 0x02, 0x03, 0x04. Получить это значение на основании состояния контактов.

https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/073d5e43f6f0fb0690a8f9b7a807439795000b26/Encoder.c#L14 Вынести из прерывания. https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/073d5e43f6f0fb0690a8f9b7a807439795000b26/Encoder.c#L15 Значение next_state должны быть одним из 0x01, 0x02, 0x03, 0x04. Получить это значение на основании состояния контактов.
stud126169 commented 2024-02-03 14:32:44 +00:00
Owner
Copy Link

Добрый вечер Степан Андреевич, загрузил последнюю версию кода проекта согласно вашим замечаниям

Добрый вечер Степан Андреевич, загрузил последнюю версию кода проекта согласно вашим замечаниям
usr22772 commented 2024-02-13 16:34:45 +00:00
Owner
Copy Link

Я совершенно не понял содержимое новой версии энкодера. Вопрос: ENC_CLK и ENC_DAT это что за контакты и какого интерфейса?

Я совершенно не понял содержимое новой версии энкодера. Вопрос: ENC_CLK и ENC_DAT это что за контакты и какого интерфейса?
stud126169 commented 2024-02-13 16:42:12 +00:00
Owner
Copy Link

У нас используется плата ArduinoATmega 328p, контакт ENC_CLK отвечает за контакт CLK энкодера, контакт ENC_DAT отвечает за контакт DAT, они оба подключены к ардуине, программа считывает комбинацию сигналов 00, 01, 10, 11 и в зависимости от их значения либо увеличивает либо уменьшает и тп

У нас используется плата ArduinoATmega 328p, контакт ENC_CLK отвечает за контакт CLK энкодера, контакт ENC_DAT отвечает за контакт DAT, они оба подключены к ардуине, программа считывает комбинацию сигналов 00, 01, 10, 11 и в зависимости от их значения либо увеличивает либо уменьшает и тп
usr22772 commented 2024-02-13 16:44:20 +00:00
Owner
Copy Link

Согласно какому даташиту или иной документации есть контакты CLK и DAT?

Согласно какому даташиту или иной документации есть контакты `CLK` и `DAT`?
stud126169 commented 2024-02-13 16:45:55 +00:00
Owner
Copy Link

Эти контакты не по даташиту, это контакты ктр мы подключаем энкодер к плате ардуино

Эти контакты не по даташиту, это контакты ктр мы подключаем энкодер к плате ардуино
stud126169 commented 2024-02-13 16:54:59 +00:00
Owner
Copy Link

В старой версии так же отсутствовали эти контакты, а значение state так же увеличивалось либо уменьшалось в зависимости от значения ктр находилось на пинах порта

В старой версии так же отсутствовали эти контакты, а значение state так же увеличивалось либо уменьшалось в зависимости от значения ктр находилось на пинах порта
stud126169 commented 2024-02-13 16:56:36 +00:00
Owner
Copy Link

Старый код:
#include <avr/interrupt.h>
#include <avr/io.h>
/* Дефайны /
#define F_CPU 1000000UL
#define PIND_MASK 0b00001100
/ Переменные /
volatile uint8_t next_state, prev_state, up_state, down_state, state;
int sw34; // Передача
/ Прерывания */
ISR(TIMER1_COMPA_vect) {
next_state = PIND & PIND_MASK;
if (next_state != prev_state) {
switch (prev_state) {
case 8: {
if (next_state == 3) state++;
if (next_state == 0) state--;
break;
}
case 0: {
if (next_state == 2) state++;
if (next_state == 1) state--;
break;
}
case 4: {
if (next_state == 0) state++;
if (next_state == 3) state--;
break;
}
case 12: {
if (next_state == 1) state++;
if (next_state == 2) state--;
break;
}
default: {
break;
}
}
prev_state = next_state; // Присваиваем текущему состоянию предыдущее
}
TCNT1H = 0x00;
TCNT1L = 0x00; // Обнуляем счетчик(на всякий случай)
}
void timer1_init() {
TCCR1A = 0x00;
TCCR1B |= (1«CS11); // устанавливаем коэффициент деления 8
TCNT1H = 0x00; // Обнуляем старший и младший байты счетного регистра
TCNT1L = 0x00;
OCR1AH = 0x03; // Настройка регистра сравнения для старшего и младщего байта
OCR1AL = 0xE8;

// Разрешаем прерывание таймера по совпадению с OCR1A
TIMSK1 |= (1«OCIE1A);
}
int main(void) {
Serial.begin(9600);
int s;
DDRD = 0x00;
PORTD = 0x0C;
timer1_init(); // Инициализация таймера
sei(); // Разрешаем глобальные прерывания
while (1) {
if (state != s)
{
Serial.println(s);
s = state;
}
}
}

Старый код: #include <avr/interrupt.h> #include <avr/io.h> /* Дефайны */ #define F_CPU 1000000UL #define PIND_MASK 0b00001100 /* Переменные */ volatile uint8_t next_state, prev_state, up_state, down_state, state; int sw34; // Передача /* Прерывания */ ISR(TIMER1_COMPA_vect) { next_state = PIND & PIND_MASK; if (next_state != prev_state) { switch (prev_state) { case 8: { if (next_state == 3) state++; if (next_state == 0) state--; break; } case 0: { if (next_state == 2) state++; if (next_state == 1) state--; break; } case 4: { if (next_state == 0) state++; if (next_state == 3) state--; break; } case 12: { if (next_state == 1) state++; if (next_state == 2) state--; break; } default: { break; } } prev_state = next_state; // Присваиваем текущему состоянию предыдущее } TCNT1H = 0x00; TCNT1L = 0x00; // Обнуляем счетчик(на всякий случай) } void timer1_init() { TCCR1A = 0x00; TCCR1B |= (1«CS11); // устанавливаем коэффициент деления 8 TCNT1H = 0x00; // Обнуляем старший и младший байты счетного регистра TCNT1L = 0x00; OCR1AH = 0x03; // Настройка регистра сравнения для старшего и младщего байта OCR1AL = 0xE8; // Разрешаем прерывание таймера по совпадению с OCR1A TIMSK1 |= (1«OCIE1A); } int main(void) { Serial.begin(9600); int s; DDRD = 0x00; PORTD = 0x0C; timer1_init(); // Инициализация таймера sei(); // Разрешаем глобальные прерывания while (1) { if (state != s) { Serial.println(s); s = state; } } }
stud126169 commented 2024-02-13 17:02:20 +00:00
Owner
Copy Link

Контакты ктр подключены к ардуино мы заранее определяем, в самом начале программы а затем снимаем с них значения, если мы нажмем на кнопку энкодера клик, то она на пине ENC_CLK выдаст единицу в момент нажатия, аналогично и с DAT_CLK, и в зависимости от комбинации этих значений, наша переменная state будет либо увеличиваться либо уменьшаться, в живой сборке работает корректно и выводит правильные значения

Контакты ктр подключены к ардуино мы заранее определяем, в самом начале программы а затем снимаем с них значения, если мы нажмем на кнопку энкодера клик, то она на пине ENC_CLK выдаст единицу в момент нажатия, аналогично и с DAT_CLK, и в зависимости от комбинации этих значений, наша переменная state будет либо увеличиваться либо уменьшаться, в живой сборке работает корректно и выводит правильные значения
usr22772 commented 2024-02-13 17:27:26 +00:00
Owner
Copy Link

Кнопка энкодера? Про какой энкодер речь идёт?

Кнопка энкодера? Про какой энкодер речь идёт?
stud126169 commented 2024-02-13 17:39:01 +00:00
Owner
Copy Link

Ой не кнопка, используется инкрементальный энкодер KY-040, просто в симуляторе на wokwi для прокрута нужно нажать на стрелочку по часовой либо против часовой, на живой сборке для прокрута нужно покрутить, на живой сборке так же выдает корректные значения в зависимости от того куда крутишь

Ой не кнопка, используется инкрементальный энкодер KY-040, просто в симуляторе на wokwi для прокрута нужно нажать на стрелочку по часовой либо против часовой, на живой сборке для прокрута нужно покрутить, на живой сборке так же выдает корректные значения в зависимости от того куда крутишь
stud126169 commented 2024-02-13 17:40:07 +00:00
Owner
Copy Link

Если нужно могу приложить скриншоты работы из wokwi и живой сборки

Если нужно могу приложить скриншоты работы из wokwi и живой сборки
stud126169 commented 2024-02-13 17:41:47 +00:00
Owner
Copy Link

На кнопках тоже работает корректно

На кнопках тоже работает корректно
stud126169 commented 2024-02-13 17:42:23 +00:00
Owner
Copy Link

Это если энкодер заменить на две кнопки

Это если энкодер заменить на две кнопки
usr22772 commented 2024-02-13 17:49:00 +00:00
Owner
Copy Link

Как долго будет процессор будет выполнять функцию int encoder() из

Encoder.c Line 8 in 52904681ef
int encoder()
, если контакт ENC_CLK будет притянут к земле всё время?

Как долго будет процессор будет выполнять функцию `int encoder()` из https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/52904681ef12a89b24a360757113ae27655bc838/Encoder.c#L8, если контакт `ENC_CLK` будет притянут к земле всё время?
stud126169 commented 2024-02-13 17:59:47 +00:00
Owner
Copy Link

Мы в Мэйне вызываем функцию энкодера в loop, то есть постоянно, если значение энкодера не менялось то он просто будет выводить старое значение бесконечно пока мы не завершим работу программы, был вариант реализации через считывание старого значения и его сравнения с новым поступившим и если они не одинаковые то выводить новое значения, но данный вариант при работе общей программы к сожалению почему-то не работает, но если вместо возвращаемого значения выводить через Serial.println, то работает как нужно и выводит только новые значения

Мы в Мэйне вызываем функцию энкодера в loop, то есть постоянно, если значение энкодера не менялось то он просто будет выводить старое значение бесконечно пока мы не завершим работу программы, был вариант реализации через считывание старого значения и его сравнения с новым поступившим и если они не одинаковые то выводить новое значения, но данный вариант при работе общей программы к сожалению почему-то не работает, но если вместо возвращаемого значения выводить через Serial.println, то работает как нужно и выводит только новые значения
stud126169 commented 2024-02-13 18:02:16 +00:00
Owner
Copy Link

А если говорить про конкретное время работы программы одного цикла, то это нужно засекать, либо в атмеле либо в симуляторе, но насколько я знаю происходит вывод очень быстро

А если говорить про конкретное время работы программы одного цикла, то это нужно засекать, либо в атмеле либо в симуляторе, но насколько я знаю происходит вывод очень быстро
usr22772 commented 2024-02-13 18:15:22 +00:00
Owner
Copy Link

Как долго будет процессор будет выполнять функцию int encoder() из

Encoder.c Line 8 in 52904681ef
int encoder()
, если контакт ENC_CLK будет притянут к земле всё время?

Это вопрос на логику. Нужно посмотреть на свой собственный код и дать ответ на вопрос.

> Как долго будет процессор будет выполнять функцию `int encoder()` из https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/52904681ef12a89b24a360757113ae27655bc838/Encoder.c#L8, если контакт `ENC_CLK` будет притянут к земле всё время? Это вопрос на логику. Нужно посмотреть на свой собственный код и дать ответ на вопрос.
stud126169 commented 2024-02-13 18:20:18 +00:00
Owner
Copy Link

понятно, прошу прощения что сам вопрос не так понял

понятно, прошу прощения что сам вопрос не так понял
stud126169 commented 2024-02-13 19:24:19 +00:00
Owner
Copy Link

Он будет нам очень быстро возвращать значение state, но так как у нас ENC_CLK подключен к земле все время, то внутрь функции он не попадет, соответственно значение state будет выводиться с большой скоростью, так как задержка тоже не будет учитываться , то он просто будет нам постоянно выводит 0 с очень большой скоростью

Он будет нам очень быстро возвращать значение state, но так как у нас ENC_CLK подключен к земле все время, то внутрь функции он не попадет, соответственно значение state будет выводиться с большой скоростью, так как задержка тоже не будет учитываться , то он просто будет нам постоянно выводит 0 с очень большой скоростью
usr22772 commented 2024-02-14 10:26:46 +00:00
Owner
Copy Link

Как долго будет процессор будет выполнять функцию int encoder() из

Encoder.c Line 8 in 52904681ef
int encoder()
, если контакт ENC_CLK будет притянут к земле всё время?

Прошу написать результаты очной встречи в рамках этого разговора

> Как долго будет процессор будет выполнять функцию `int encoder()` из https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/52904681ef12a89b24a360757113ae27655bc838/Encoder.c#L8, если контакт `ENC_CLK` будет притянут к земле всё время? Прошу написать результаты очной встречи в рамках этого разговора
stud126169 commented 2024-02-15 08:07:07 +00:00
Owner
Copy Link

Результатами очной встречи стали доработки по энкодера, нужно будет переписать старый код, чтобы next_state всегда было равно 0, 1, 2, 3 и загрузить измененную версию кода в GIT, данный момент был сделан и код в Git загружен, буду ждать ваших комментариев по его реализации, так же нужно будет переделать задержку у датчика температуры и влажности, чтобы код не виснул при его работе, сделать это через таймер и всё

Результатами очной встречи стали доработки по энкодера, нужно будет переписать старый код, чтобы next_state всегда было равно 0, 1, 2, 3 и загрузить измененную версию кода в GIT, данный момент был сделан и код в Git загружен, буду ждать ваших комментариев по его реализации, так же нужно будет переделать задержку у датчика температуры и влажности, чтобы код не виснул при его работе, сделать это через таймер и всё
usr22772 commented 2024-02-15 16:52:37 +00:00
Owner
Copy Link

Encoder.c Line 4 in 40c876d282
#define PIND_MASK 0b11000011

Ну а если будет так что один сигнал энкодера будет поступать на конткт D2, а другой D6?

https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/40c876d282257078f72d43c4551ab7d7a3dc4b56/Encoder.c#L4 Ну а если будет так что один сигнал энкодера будет поступать на конткт D2, а другой D6?
stud126169 commented 2024-02-15 16:57:44 +00:00
Owner
Copy Link

На этот случай тоже есть возможность реализации, можно сделать такое же смещение в зависимости от контактов, просто сами подключенные контакты а именно PIND_MASK разбить на два Пина, ктр будут отвечать за контакты энкодера, и так же их сдвигать в зависимости от полученное значения пинов, просто данный вариант показывает саму идею реализации, если она подойдет, то я ее доработаю до автоматизма

На этот случай тоже есть возможность реализации, можно сделать такое же смещение в зависимости от контактов, просто сами подключенные контакты а именно PIND_MASK разбить на два Пина, ктр будут отвечать за контакты энкодера, и так же их сдвигать в зависимости от полученное значения пинов, просто данный вариант показывает саму идею реализации, если она подойдет, то я ее доработаю до автоматизма
usr22772 commented 2024-02-16 12:52:49 +00:00
Owner
Copy Link

Именно так и сделать

Именно так и сделать
stud126169 commented 2024-02-16 12:55:09 +00:00
Owner
Copy Link

Понял, в таком случае я автоматизирую полностью, то что я написал и после загружу обновленную версию кода в Git и сброшу вам ссылку

Понял, в таком случае я автоматизирую полностью, то что я написал и после загружу обновленную версию кода в Git и сброшу вам ссылку
usr22772 commented 2024-02-18 11:07:07 +00:00
Owner
Copy Link

Encoder.c Line 14 in f3ee7efb61
PIND_SHIFT_1 = 0;

Вместо портянки с 14-й по 150-ю строку можно просто написать вверху

#define PIND_SHIFT_1 (4)
#define PIND_SHIFT_2 (6)
#define PIND_CLK (1 << PIND_SHIFT_1)
#define PIND_DT (1 << PIND_SHIFT_2 )
https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f3ee7efb611d57c2b9b9596bc46eef50b1837394/Encoder.c#L14 Вместо портянки с 14-й по 150-ю строку можно просто написать вверху ``` #define PIND_SHIFT_1 (4) #define PIND_SHIFT_2 (6) #define PIND_CLK (1 << PIND_SHIFT_1) #define PIND_DT (1 << PIND_SHIFT_2 ) ```
usr22772 commented 2024-02-18 11:07:48 +00:00
Owner
Copy Link

Encoder.c Line 158 in f3ee7efb61
next_state = (PIND & PIND_MASK);

Пробовали? Работает?

https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f3ee7efb611d57c2b9b9596bc46eef50b1837394/Encoder.c#L158 Пробовали? Работает?
stud126169 commented 2024-02-18 11:10:05 +00:00
Owner
Copy Link

Да в эмуляторе пробовал все работает, нужно только указать значения пинов в самом начале и все, все комбинации в ручную вводил и проверял, в живую буду сегодня проверять

Да в эмуляторе пробовал все работает, нужно только указать значения пинов в самом начале и все, все комбинации в ручную вводил и проверял, в живую буду сегодня проверять
stud126169 commented 2024-02-18 11:14:15 +00:00
Owner
Copy Link

Encoder.c Line 14 in f3ee7efb61
PIND_SHIFT_1 = 0;

Вместо портянки с 14-й по 150-ю строку можно просто написать вверху

#define PIND_SHIFT_1 (4)
#define PIND_SHIFT_2 (6)
#define PIND_CLK (1 << PIND_SHIFT_1)
#define PIND_DT (1 << PIND_SHIFT_2 )

Попробую сегодня и этот вариант, просто вариант реализации ктр я сейчас написал кажется более гибким, так как я конкретно под каждую комбинацию пинов могу задать нужное смещение для каждого пина

> https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f3ee7efb611d57c2b9b9596bc46eef50b1837394/Encoder.c#L14 > > Вместо портянки с 14-й по 150-ю строку можно просто написать вверху > > ``` > #define PIND_SHIFT_1 (4) > #define PIND_SHIFT_2 (6) > #define PIND_CLK (1 << PIND_SHIFT_1) > #define PIND_DT (1 << PIND_SHIFT_2 ) > ``` Попробую сегодня и этот вариант, просто вариант реализации ктр я сейчас написал кажется более гибким, так как я конкретно под каждую комбинацию пинов могу задать нужное смещение для каждого пина
usr22772 commented 2024-02-18 11:15:01 +00:00
Owner
Copy Link

Прошу подумать хорошо над строкой кода. Опишите в комментариях каждую строчку кода так, чтобы понял человек, который не работал с энкодерами. Читаем состояние энкодера не является хорошим комментарием.

    PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1;
    PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2;
    PIND_MASK = (PIN_CLK_2 + PIN_DT_2);

    // Читаем состояние энкодера
    next_state = (PIND & PIND_MASK);

    // Обновляем состояние энкодера
    if (next_state != prev_state)
    {
Прошу подумать хорошо над строкой кода. Опишите в комментариях каждую строчку кода так, чтобы понял человек, который не работал с энкодерами. _Читаем состояние энкодера_ не является хорошим комментарием. ``` PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1; PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2; PIND_MASK = (PIN_CLK_2 + PIN_DT_2); // Читаем состояние энкодера next_state = (PIND & PIND_MASK); // Обновляем состояние энкодера if (next_state != prev_state) { ```
usr22772 commented 2024-02-18 11:17:57 +00:00
Owner
Copy Link

Попробую сегодня и этот вариант, просто вариант реализации ктр я сейчас написал кажется более гибким, так как я конкретно под каждую комбинацию пинов могу задать нужное смещение для каждого пина

Ну ведь в дефайнах и задаётся однозначно нужное смещение для каждого пина. Больше ничего делать не надо. Ваши вычисления на 130 строк компилятор оптимизирует до этих же 4х констант.

> Попробую сегодня и этот вариант, просто вариант реализации ктр я сейчас написал кажется более гибким, так как я конкретно под каждую комбинацию пинов могу задать нужное смещение для каждого пина Ну ведь в дефайнах и задаётся однозначно нужное смещение для каждого пина. Больше ничего делать не надо. Ваши вычисления на 130 строк компилятор оптимизирует до этих же 4х констант.
stud126169 commented 2024-02-18 11:21:53 +00:00
Owner
Copy Link

Я понял, просто я хотел максимально автоматизировать работу чтобы в define менять как можно меньше переменных, поэтому реализовал это данным способом

Я понял, просто я хотел максимально автоматизировать работу чтобы в define менять как можно меньше переменных, поэтому реализовал это данным способом
stud126169 commented 2024-02-18 11:23:54 +00:00
Owner
Copy Link

Я могу изменить и убрать эти вычисления о ктр вы говорите и сделать все в дефайн, но тогда нужно будет каждый раз указывать нужное смещение и пины для работы программы, сейчас указываются только подключенные пины и на основании этих пинов программа автоматом выдает нужное смещение для них

Я могу изменить и убрать эти вычисления о ктр вы говорите и сделать все в дефайн, но тогда нужно будет каждый раз указывать нужное смещение и пины для работы программы, сейчас указываются только подключенные пины и на основании этих пинов программа автоматом выдает нужное смещение для них
stud126169 commented 2024-02-18 11:31:09 +00:00
Owner
Copy Link

Прошу подумать хорошо над строкой кода. Опишите в комментариях каждую строчку кода так, чтобы понял человек, который не работал с энкодерами. Читаем состояние энкодера не является хорошим комментарием.

    PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1;
    PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2;
    PIND_MASK = (PIN_CLK_2 + PIN_DT_2);

    // Читаем состояние энкодера
    next_state = (PIND & PIND_MASK);

    // Обновляем состояние энкодера
    if (next_state != prev_state)
    {

Понял, сделаю более подробные комментарии для пользователя

> Прошу подумать хорошо над строкой кода. Опишите в комментариях каждую строчку кода так, чтобы понял человек, который не работал с энкодерами. _Читаем состояние энкодера_ не является хорошим комментарием. > > ``` > PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1; > PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2; > PIND_MASK = (PIN_CLK_2 + PIN_DT_2); > > // Читаем состояние энкодера > next_state = (PIND & PIND_MASK); > > // Обновляем состояние энкодера > if (next_state != prev_state) > { > ``` Понял, сделаю более подробные комментарии для пользователя
usr22772 commented 2024-02-18 12:06:50 +00:00
Owner
Copy Link

Не для пользователя. Для себя и меня. Вам нужно самому понять, что тут написано.

Не для пользователя. Для себя и меня. Вам нужно самому понять, что тут написано.
stud126169 commented 2024-02-18 12:07:51 +00:00
Owner
Copy Link

Не для пользователя. Для себя и меня. Вам нужно самому понять, что тут написано.

Понял

> Не для пользователя. Для себя и меня. Вам нужно самому понять, что тут написано. Понял
stud126169 commented 2024-02-18 12:10:15 +00:00
Owner
Copy Link

Нужно ли ещё что-нибудь доработать по коду?

Нужно ли ещё что-нибудь доработать по коду?
usr22772 commented 2024-02-20 09:30:26 +00:00
Owner
Copy Link

Вы проверяли? Работает? Подумайте!

// Определяем бит в регистре PIND для сигнала с клока энкодера.
#define PIND_CLK 0b00010000
// Определяем бит в регистре PIND для сигнала с DT энкодера.
#define PIND_DT 0b01000000
// Определяем сдвиг для получения значения сигнала с клока энкодера.
#define PIND_SHIFT_1 (4)
// Определяем сдвиг для получения значения сигнала с DT энкодера.
#define PIND_SHIFT_2 (5)

    // Значение переменной PIN_CLK_2 равно результату побитовой операции "И" между PIND и PIND_CLK с последующим сдвигом на PIND_SHIFT_1 бит вправо
    PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1;
    // Значение переменной PIN_DT_2 равно результату побитовой операции "И" между PIND и PIND_DT с последующим сдвигом на PIND_SHIFT_2 бит вправо
    PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2;
    // Значение переменной PIND_MASK равно сумме значений переменных PIN_CLK_2 и PIN_DT_2
    PIND_MASK = (PIN_CLK_2 + PIN_DT_2);
Вы проверяли? Работает? Подумайте! ``` // Определяем бит в регистре PIND для сигнала с клока энкодера. #define PIND_CLK 0b00010000 // Определяем бит в регистре PIND для сигнала с DT энкодера. #define PIND_DT 0b01000000 // Определяем сдвиг для получения значения сигнала с клока энкодера. #define PIND_SHIFT_1 (4) // Определяем сдвиг для получения значения сигнала с DT энкодера. #define PIND_SHIFT_2 (5) ``` ``` // Значение переменной PIN_CLK_2 равно результату побитовой операции "И" между PIND и PIND_CLK с последующим сдвигом на PIND_SHIFT_1 бит вправо PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1; // Значение переменной PIN_DT_2 равно результату побитовой операции "И" между PIND и PIND_DT с последующим сдвигом на PIND_SHIFT_2 бит вправо PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2; // Значение переменной PIND_MASK равно сумме значений переменных PIN_CLK_2 и PIN_DT_2 PIND_MASK = (PIN_CLK_2 + PIN_DT_2); ```
usr22772 commented 2024-02-20 09:32:36 +00:00
Owner
Copy Link

Encoder.c Line 14 in b77eb4bf4f
volatile uint8_t next_state, prev_state, state, state_1;

что значит volatile?

Encoder.c Line 15 in b77eb4bf4f
int sw34, PIND_MASK, PIN_CLK_2, PIN_DT_2;

PIND_MASK, PIN_CLK_2, PIN_DT_2
В верхнем регистре как правило только дефайны прописываются. Переменные в нижнем регистре. Банальный Си-шный кодстайл.

https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/b77eb4bf4f154b4f41681a384f0a495acfd43966/Encoder.c#L14 что значит `volatile`? https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/b77eb4bf4f154b4f41681a384f0a495acfd43966/Encoder.c#L15 `PIND_MASK, PIN_CLK_2, PIN_DT_2` В верхнем регистре как правило только дефайны прописываются. Переменные в нижнем регистре. Банальный Си-шный кодстайл.
stud126169 commented 2024-02-20 09:42:49 +00:00
Owner
Copy Link

Volatile это квалификатор переменной, ктр просит чтобы компилятор брал значение сразу из оперативной памяти, а не из регистра хранения, насколько я знаю это позволяет убрать неточность, которая может возникнуть в регистрах хранения переменной, насчет регистров и их объявления, поменяю, нужны ли будут какие-либо еще изменения по коду и доработки, или можно будет переписать пз по одобренному вами коду и сбросить вам?

Volatile это квалификатор переменной, ктр просит чтобы компилятор брал значение сразу из оперативной памяти, а не из регистра хранения, насколько я знаю это позволяет убрать неточность, которая может возникнуть в регистрах хранения переменной, насчет регистров и их объявления, поменяю, нужны ли будут какие-либо еще изменения по коду и доработки, или можно будет переписать пз по одобренному вами коду и сбросить вам?
stud126169 commented 2024-02-20 09:47:12 +00:00
Owner
Copy Link

Вы проверяли? Работает? Подумайте!

// Определяем бит в регистре PIND для сигнала с клока энкодера.
#define PIND_CLK 0b00010000
// Определяем бит в регистре PIND для сигнала с DT энкодера.
#define PIND_DT 0b01000000
// Определяем сдвиг для получения значения сигнала с клока энкодера.
#define PIND_SHIFT_1 (4)
// Определяем сдвиг для получения значения сигнала с DT энкодера.
#define PIND_SHIFT_2 (5)

    // Значение переменной PIN_CLK_2 равно результату побитовой операции "И" между PIND и PIND_CLK с последующим сдвигом на PIND_SHIFT_1 бит вправо
    PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1;
    // Значение переменной PIN_DT_2 равно результату побитовой операции "И" между PIND и PIND_DT с последующим сдвигом на PIND_SHIFT_2 бит вправо
    PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2;
    // Значение переменной PIND_MASK равно сумме значений переменных PIN_CLK_2 и PIN_DT_2
    PIND_MASK = (PIN_CLK_2 + PIN_DT_2);

Сам код проверял, все работает, в случае изменения подключения, мы меняем значения сдвига и пины подключения и всё работает как нужно, проверил на нескольких комбинациях и в живую и в симуляторе работает корректно

> Вы проверяли? Работает? Подумайте! > ``` > // Определяем бит в регистре PIND для сигнала с клока энкодера. > #define PIND_CLK 0b00010000 > // Определяем бит в регистре PIND для сигнала с DT энкодера. > #define PIND_DT 0b01000000 > // Определяем сдвиг для получения значения сигнала с клока энкодера. > #define PIND_SHIFT_1 (4) > // Определяем сдвиг для получения значения сигнала с DT энкодера. > #define PIND_SHIFT_2 (5) > ``` > > ``` > // Значение переменной PIN_CLK_2 равно результату побитовой операции "И" между PIND и PIND_CLK с последующим сдвигом на PIND_SHIFT_1 бит вправо > PIN_CLK_2 = (PIND & PIND_CLK) >> PIND_SHIFT_1; > // Значение переменной PIN_DT_2 равно результату побитовой операции "И" между PIND и PIND_DT с последующим сдвигом на PIND_SHIFT_2 бит вправо > PIN_DT_2 = (PIND & PIND_DT) >> PIND_SHIFT_2; > // Значение переменной PIND_MASK равно сумме значений переменных PIN_CLK_2 и PIN_DT_2 > PIND_MASK = (PIN_CLK_2 + PIN_DT_2); > ``` Сам код проверял, все работает, в случае изменения подключения, мы меняем значения сдвига и пины подключения и всё работает как нужно, проверил на нескольких комбинациях и в живую и в симуляторе работает корректно
usr22772 commented 2024-02-20 09:53:13 +00:00
Owner
Copy Link

Скриншот эмулятора сюда

Скриншот эмулятора сюда
stud126169 commented 2024-02-20 09:56:38 +00:00
Owner
Copy Link

Разумеется, сейчас сброшу

Разумеется, сейчас сброшу
stud126169 commented 2024-02-20 09:59:25 +00:00
Owner
Copy Link

image

![image](/attachments/d1d3926b-8aa1-4c83-a85f-1fd27c34073c)
image.png
294 KiB
stud126169 commented 2024-02-20 10:01:13 +00:00
Owner
Copy Link

image

![image](/attachments/c6c2ef9a-2a9c-4318-b730-ca263e21b2b5)
image.png
292 KiB
stud126169 commented 2024-02-20 10:02:09 +00:00
Owner
Copy Link

image

![image](/attachments/5068cff6-deb4-4bbf-a52e-f06573988d52)
image.png
290 KiB
stud126169 commented 2024-02-20 10:02:50 +00:00
Owner
Copy Link

Скриншоты с тремя разными комбинациями

Скриншоты с тремя разными комбинациями
stud126169 commented 2024-02-20 10:02:59 +00:00
Owner
Copy Link

Выдаёт значения корректно

Выдаёт значения корректно
usr22772 commented 2024-02-20 10:09:52 +00:00
Owner
Copy Link

Переделать схему на два переключателя. А то не ясно, что у вас там происходит.

Переделать схему на два переключателя. А то не ясно, что у вас там происходит.
stud126169 commented 2024-02-20 10:10:17 +00:00
Owner
Copy Link

имеете в виду поставить кнопки вместо энкодера?

имеете в виду поставить кнопки вместо энкодера?
usr22772 commented 2024-02-20 10:11:49 +00:00
Owner
Copy Link

Переключатели.

image

Переключатели. ![image](/attachments/e1237499-96fe-49ef-89fd-a793233dee72)
image.png
3.1 KiB
stud126169 commented 2024-02-20 10:14:56 +00:00
Owner
Copy Link

image

![image](/attachments/bcecdc8b-92dd-4473-b35a-c583ec8b7a6e)
image.png
280 KiB
usr22772 commented 2024-02-20 11:02:05 +00:00
Owner
Copy Link

Ну даже на скрине смотрю на код и вижу что что-то не так.

Ну даже на скрине смотрю на код и вижу что что-то не так.
usr22772 commented 2024-02-20 11:04:30 +00:00
Owner
Copy Link

Понял! Но это так не очевидно! Вы сразу сдвигаете на позицию в бит 1. Не надо так.

Понял! Но это так не очевидно! Вы сразу сдвигаете на позицию в бит 1. Не надо так.
usr22772 commented 2024-02-20 11:06:05 +00:00
Owner
Copy Link

Encoder.c Line 40 in b77eb4bf4f
case 0: // CW

тут необходимо ещё раз посмотреть и подумать.

https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/b77eb4bf4f154b4f41681a384f0a495acfd43966/Encoder.c#L40 тут необходимо ещё раз посмотреть и подумать.
stud126169 commented 2024-02-20 11:06:16 +00:00
Owner
Copy Link

Аааааа, понял

Аааааа, понял
stud126169 commented 2024-02-20 11:07:11 +00:00
Owner
Copy Link

Я тоже подумаю и посмотрю как это можно по другому реализовать

Я тоже подумаю и посмотрю как это можно по другому реализовать
stud126169 commented 2024-02-20 11:08:51 +00:00
Owner
Copy Link

Просто у меня появилась идея как реализовать то о чем вы просили, это идея как раз таки и заключалась в смещении на нужное кол-во битов, чтобы мы всегда получали либо 0 либо 1 либо 2 либо 3, раз вы говорите что так нельзя делать, то я подумаю как еще можно этот момент реализовать, но пока что если честно конкретных идей в голову не приходит

Просто у меня появилась идея как реализовать то о чем вы просили, это идея как раз таки и заключалась в смещении на нужное кол-во битов, чтобы мы всегда получали либо 0 либо 1 либо 2 либо 3, раз вы говорите что так нельзя делать, то я подумаю как еще можно этот момент реализовать, но пока что если честно конкретных идей в голову не приходит
stud126169 commented 2024-02-20 11:12:10 +00:00
Owner
Copy Link

И по итогу мы это и получаем с любой комбинации подключений у нас всегда будет либо 00, либо 10, либо 01, либо 11

И по итогу мы это и получаем с любой комбинации подключений у нас всегда будет либо 00, либо 10, либо 01, либо 11
usr22772 commented 2024-02-23 13:01:00 +00:00
Owner
Copy Link

Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода

Encoder.c Line 26 in f75998e479
encoder_state = (pin_clk_2 << 1) | pin_dt_2;

Столбцы

  • encoder_state
  • prev_encoder_state
  • state до выполнения (пусть всегда будет "1")
  • state, которое ожидаем увидеть
  • state, которое получили после выполнения.

Подсказка: если нет желания устанавливать компилятор для хоста, то можно воспользоваться https://www.onlinegdb.com/online_c_compiler

Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f75998e4796bcb610f25b8c3ef278597895bf9ff/Encoder.c#L26 Столбцы - encoder_state - prev_encoder_state - state до выполнения (пусть всегда будет "1") - state, которое ожидаем увидеть - state, которое получили после выполнения. Подсказка: если нет желания устанавливать компилятор для хоста, то можно воспользоваться https://www.onlinegdb.com/online_c_compiler
stud126169 commented 2024-02-23 13:05:00 +00:00
Owner
Copy Link

Так я так и сделал и прогнал все значения и заметил что были перепутаны комбинации для поворота по часовой и против часовой стрелки и в результате выводились другие значения, и я исправил этот момент, сейчас сделаю таблицу со значения еще раз и после отпишу вам

Так я так и сделал и прогнал все значения и заметил что были перепутаны комбинации для поворота по часовой и против часовой стрелки и в результате выводились другие значения, и я исправил этот момент, сейчас сделаю таблицу со значения еще раз и после отпишу вам
stud126169 commented 2024-02-23 13:06:08 +00:00
Owner
Copy Link

Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода

Encoder.c Line 26 in f75998e479
encoder_state = (pin_clk_2 << 1) | pin_dt_2;

Столбцы

  • encoder_state
  • prev_encoder_state
  • state до выполнения (пусть всегда будет "1")
  • state, которое ожидаем увидеть
  • state, которое получили после выполнения.

Подсказка: если нет желания устанавливать компилятор для хоста, то можно воспользоваться https://www.onlinegdb.com/online_c_compiler

Понял, обязательно воспользуюсь, спасибо большое)

> Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода > > https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f75998e4796bcb610f25b8c3ef278597895bf9ff/Encoder.c#L26 > > Столбцы > - encoder_state > - prev_encoder_state > - state до выполнения (пусть всегда будет "1") > - state, которое ожидаем увидеть > - state, которое получили после выполнения. > > Подсказка: если нет желания устанавливать компилятор для хоста, то можно воспользоваться https://www.onlinegdb.com/online_c_compiler Понял, обязательно воспользуюсь, спасибо большое)
stud126169 commented 2024-02-25 17:34:15 +00:00
Owner
Copy Link

Степан Андреевич, я так и сделал, прогнал все значения через прошлый код и записал их как в таблице в итоге вышло следующее: Начальное положение: encoder_state = 3 prev_encoder_state = 0 state(ож) = 1 state(пол) = 0
При повороте по часовой стрелке: encoder_state = 1 prev_encoder_state = 3 state(ож) = 2 state(пол) = 1
При повороте против часовой стрелке: encoder_state = 2 prev_encoder_state = 1 state(ож) = 1 state(пол) = 1
При повороте против часовой стрелке(2 раза): encoder_state = 2 prev_encoder_state = 1 state(ож) = 0 state(пол) = 1
При повороте по часовой стрелке(2 раза): encoder_state = 2 prev_encoder_state = 1 state(ож) = 2 state(пол) = 0, значения выводил через debug программы, после этого произвёл доработки чтобы исправить на нужные мне значения, по итогу получилось исправить их, а конкретнее была исправленна ошибка со значениями при начальным положением и с переключением при увеличении и при уменьшении, я проганяю дальше значения при нынешней версии кода и он выводит нужный мне state, тот ктр я ожидаю увидеть, я просто не понимаю что нужно исправить по коду, не могли бы вы подсказать, я и с осцилограммой сигнала смотрел, и значения 2 прогонял, если ошибка в том что я сдвигаю на 1 пин pin_clk2 и не сдвигаю pin_dt2, то это всё можно подкоректировать в зависимости от того какие пины подключены

Степан Андреевич, я так и сделал, прогнал все значения через прошлый код и записал их как в таблице в итоге вышло следующее: Начальное положение: encoder_state = 3 prev_encoder_state = 0 state(ож) = 1 state(пол) = 0 При повороте по часовой стрелке: encoder_state = 1 prev_encoder_state = 3 state(ож) = 2 state(пол) = 1 При повороте против часовой стрелке: encoder_state = 2 prev_encoder_state = 1 state(ож) = 1 state(пол) = 1 При повороте против часовой стрелке(2 раза): encoder_state = 2 prev_encoder_state = 1 state(ож) = 0 state(пол) = 1 При повороте по часовой стрелке(2 раза): encoder_state = 2 prev_encoder_state = 1 state(ож) = 2 state(пол) = 0, значения выводил через debug программы, после этого произвёл доработки чтобы исправить на нужные мне значения, по итогу получилось исправить их, а конкретнее была исправленна ошибка со значениями при начальным положением и с переключением при увеличении и при уменьшении, я проганяю дальше значения при нынешней версии кода и он выводит нужный мне state, тот ктр я ожидаю увидеть, я просто не понимаю что нужно исправить по коду, не могли бы вы подсказать, я и с осцилограммой сигнала смотрел, и значения 2 прогонял, если ошибка в том что я сдвигаю на 1 пин pin_clk2 и не сдвигаю pin_dt2, то это всё можно подкоректировать в зависимости от того какие пины подключены
usr22772 commented 2024-02-25 17:38:34 +00:00
Owner
Copy Link

Мы на прошедшей встрече договорились
будет сделана таблица

> Мы на прошедшей встрече договорились > будет сделана таблица
usr22772 commented 2024-02-27 10:17:25 +00:00
Owner
Copy Link

Мы на прошедшей встрече договорились о том, что будет сделана таблица

#3 (comment)

Я более проверять не буду, пока не будет выполнена договорённость. Я не буду проверять работу чаще, чем раз в два дня. Игнорирование требований не приводит ни к чему хорошему.

> Мы на прошедшей встрече договорились о том, что будет сделана таблица https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/issues/3#issuecomment-379 Я более проверять не буду, пока не будет выполнена договорённость. Я не буду проверять работу чаще, чем раз в два дня. Игнорирование требований не приводит ни к чему хорошему.
usr22772 commented 2024-02-27 10:19:18 +00:00
Owner
Copy Link

И читайте теорию энкодеров. На экзамене буду досконально спрашивать помимо тестов.

И читайте теорию энкодеров. На экзамене буду досконально спрашивать помимо тестов.
stud126169 commented 2024-02-27 10:57:43 +00:00
Owner
Copy Link

Степан Андреевич, в данной версии кода отсутствует какая-либо ошибка влияющая на правильный вывод данных, сам код работает корректно и в живой сборке и в эмуляторе, а таблицу я вам сбрасывал значения из нее, вы сказали сделать таблицу я ее сделал, сбросил вам значения, но они остались без ответа

Степан Андреевич, в данной версии кода отсутствует какая-либо ошибка влияющая на правильный вывод данных, сам код работает корректно и в живой сборке и в эмуляторе, а таблицу я вам сбрасывал значения из нее, вы сказали сделать таблицу я ее сделал, сбросил вам значения, но они остались без ответа
stud126169 commented 2024-02-27 11:02:08 +00:00
Owner
Copy Link

И читайте теорию энкодеров. На экзамене буду досконально спрашивать помимо тестов.

Хорошо

> И читайте теорию энкодеров. На экзамене буду досконально спрашивать помимо тестов. Хорошо
stud126169 commented 2024-02-27 11:20:17 +00:00
Owner
Copy Link

Вот скриншоты работы нынешней версии кода:

Вот скриншоты работы нынешней версии кода:
4CF2C666-1EBA-4D06-97EF-C37FAF63D818.jpeg
5.9 MiB
BE73D1CB-7E49-414E-A4FE-96D96E81F76E.jpeg
5.7 MiB
FB4F908D-ABFB-482E-9D8A-975F60EA0654.jpeg
5.8 MiB
9F1CBCB1-343E-4C6D-AF36-F02377C0805C.jpeg
5.9 MiB
98BFAD52-4E34-452E-980B-AAC83A67EEF3.jpeg
5.9 MiB
4CF2C666-1EBA-4D06-97EF-C37FAF63D818.jpeg BE73D1CB-7E49-414E-A4FE-96D96E81F76E.jpeg FB4F908D-ABFB-482E-9D8A-975F60EA0654.jpeg 9F1CBCB1-343E-4C6D-AF36-F02377C0805C.jpeg 98BFAD52-4E34-452E-980B-AAC83A67EEF3.jpeg
usr22772 commented 2024-02-27 19:07:40 +00:00
Owner
Copy Link

вы сказали сделать таблицу я ее сделал

Прошу ссылку на таблицу

А лучше разместить её здесь

> вы сказали сделать таблицу я ее сделал Прошу ссылку на таблицу А лучше разместить её здесь
stud126169 commented 2024-03-01 17:44:27 +00:00
Owner
Copy Link

Переделал код согласно новой составленной таблице, в файле есть как новые значения из таблицы, так и те ктр были до изменения

Переделал код согласно новой составленной таблице, в файле есть как новые значения из таблицы, так и те ктр были до изменения
Таблица истинности энкодера.xlsx
100 KiB
usr22772 commented 2024-03-06 10:43:03 +00:00
Owner
Copy Link

Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода

Encoder.c Line 26 in f75998e479
encoder_state = (pin_clk_2 << 1) | pin_dt_2;

Столбцы

  • encoder_state
  • prev_encoder_state
  • state до выполнения (пусть всегда будет "1")
  • state, которое ожидаем увидеть
  • state, которое получили после выполнения.

Читайте и выполняйте. Берите карандаш и записывайте каждый проход по коду, буквально интерпретируйте свой код. Искренне не понимаю, в чём сложность выполнить такое простое задание.

Чего добились, сделав таблицу Таблица истинности энкодера.xlsx?

> Мы на прошедшей встрече договорились о том, что будет сделана таблица для переменных вот этого куска кода с 26й по 58ю строку кода > > https://git.vyatsu.ru/ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov/src/commit/f75998e4796bcb610f25b8c3ef278597895bf9ff/Encoder.c#L26 > > Столбцы > - encoder_state > - prev_encoder_state > - state до выполнения (пусть всегда будет "1") > - state, которое ожидаем увидеть > - state, которое получили после выполнения. Читайте и выполняйте. Берите карандаш и записывайте каждый проход по коду, буквально интерпретируйте свой код. Искренне не понимаю, в чём сложность выполнить такое простое задание. Чего добились, сделав таблицу `Таблица истинности энкодера.xlsx`?
stud126169 commented 2024-03-06 13:27:59 +00:00
Owner
Copy Link

Степан Андреевич добрый вечер, я сделал таблицу согласно вашим замечаниям, по старому коду и прогнал все значения через debug как показано в таблице, а именно: encoder_state, prev_encoder_state и state, не написал только state(ожидаемый), но он должен быть таким как в таблице сверху, я сделал ровным счетом то что вы сказали, а именно через debug прогнал все значения и записал их в таблицу и увидел что в старом коде значение state меняется один раз в то время когда при повороте должна меняться 4 раза и я переделал код согласно этому замечанию, добавив переменную total_state ктр увеличивается на 1 или уменьшается в зависимости от значения state, само значение state в конце цикла обнуляется, единственное что сделано некорректно так это начальное состояние энкодера, ктр должно быть 0 0, так как я ставлю сразу 1 в define, но этот момент я исправил присвоив начальное значение prev_encoder_state равное 3, и у меня отличается от таблицы только начальное состояние а именно оно будет не 0 0, а 3 3, но это не влияет никоим образом на работу энкодера, так как при повороте он так же проходит все 4 значения и либо делает state равным 4 либо делает его равным -4 и после в зависимости от этого значения к total_state либо прибавляется 1 либо вычетается

Степан Андреевич добрый вечер, я сделал таблицу согласно вашим замечаниям, по старому коду и прогнал все значения через debug как показано в таблице, а именно: encoder_state, prev_encoder_state и state, не написал только state(ожидаемый), но он должен быть таким как в таблице сверху, я сделал ровным счетом то что вы сказали, а именно через debug прогнал все значения и записал их в таблицу и увидел что в старом коде значение state меняется один раз в то время когда при повороте должна меняться 4 раза и я переделал код согласно этому замечанию, добавив переменную total_state ктр увеличивается на 1 или уменьшается в зависимости от значения state, само значение state в конце цикла обнуляется, единственное что сделано некорректно так это начальное состояние энкодера, ктр должно быть 0 0, так как я ставлю сразу 1 в define, но этот момент я исправил присвоив начальное значение prev_encoder_state равное 3, и у меня отличается от таблицы только начальное состояние а именно оно будет не 0 0, а 3 3, но это не влияет никоим образом на работу энкодера, так как при повороте он так же проходит все 4 значения и либо делает state равным 4 либо делает его равным -4 и после в зависимости от этого значения к total_state либо прибавляется 1 либо вычетается
usr22772 commented 2024-03-06 15:32:29 +00:00
Owner
Copy Link

не написал только state(ожидаемый),

я сделал ровным счетом то что вы сказали,

нет, вы не сделали.

> не написал только state(ожидаемый), > я сделал ровным счетом то что вы сказали, нет, вы не сделали.
usr22772 commented 2024-03-06 15:37:46 +00:00
Owner
Copy Link

Я не говорю, что код ТЕПЕРЬ не рабочий. Но я говорю, что надо бы разочек выполнить простые действия. И не писать много текста, а сделать таблицу на 5 столбцов.

Я не говорю, что код ТЕПЕРЬ не рабочий. Но я говорю, что надо бы разочек выполнить простые действия. И не писать много текста, а сделать таблицу на 5 столбцов.
usr22772 commented 2024-03-06 15:39:05 +00:00
Owner
Copy Link

И куда делся Encoder.c? отсюда вопрос: чем отличаются C(CPP) файлы от H(HPP) файлов?

И куда делся `Encoder.c`? отсюда вопрос: чем отличаются C(CPP) файлы от H(HPP) файлов?
stud126169 commented 2024-03-06 16:23:00 +00:00
Owner
Copy Link

И куда делся Encoder.c? отсюда вопрос: чем отличаются C(CPP) файлы от H(HPP) файлов?

Просто спешили и сделали сразу заголовочные файлы чтобы показать вам в прошлую встречу что оба кода корректно работают в main при живой сборке, если говорить в чем отличие, то отличие между ними в том что в CPP файле хранится исходный код программы в то время как в H файле находиться ссылки на функции в файле CPP, если говорить примером, то в моем понимании CPP файл что-то вроде книги в которой все написано, а H файл является оглавлением для этой книги

> И куда делся `Encoder.c`? отсюда вопрос: чем отличаются C(CPP) файлы от H(HPP) файлов? Просто спешили и сделали сразу заголовочные файлы чтобы показать вам в прошлую встречу что оба кода корректно работают в main при живой сборке, если говорить в чем отличие, то отличие между ними в том что в CPP файле хранится исходный код программы в то время как в H файле находиться ссылки на функции в файле CPP, если говорить примером, то в моем понимании CPP файл что-то вроде книги в которой все написано, а H файл является оглавлением для этой книги
stud126169 commented 2024-03-06 16:24:29 +00:00
Owner
Copy Link

не написал только state(ожидаемый),

я сделал ровным счетом то что вы сказали,

нет, вы не сделали.

Хорошо Степан Андреевич, я еще раз прогоню все значения кода через debug и составлю таблицу со значениями и приложу ее сюда

> > не написал только state(ожидаемый), > > > я сделал ровным счетом то что вы сказали, > > нет, вы не сделали. Хорошо Степан Андреевич, я еще раз прогоню все значения кода через debug и составлю таблицу со значениями и приложу ее сюда
stud126169 commented 2024-03-06 16:26:17 +00:00
Owner
Copy Link

А саму структуру общего проекта мы переделаем, как только получим ваше одобрение, вернем CPP файлы и сделаем правильные заголовочный файлы для мейна

А саму структуру общего проекта мы переделаем, как только получим ваше одобрение, вернем CPP файлы и сделаем правильные заголовочный файлы для мейна
usr22772 commented 2024-03-06 20:04:36 +00:00
Owner
Copy Link
      if ( state ==  4 )
      { 
        total_state++;
        state=0;
      }
      if ( state == -4 )
      { 
        total_state--;
        state=0;
      }

Зачем вот это я не понимаю, это не нужно, у вас есть переменная state, которая меняется каждый раз при изменении значений контактов. В остальном верно. Читайте теорию энкодеров, учите. В большинстве современных контроллерах блок, обрабатывающий сигналы с инкрементального энекодера реализованы аппаратно в таймере или в отдельном блоке, и на каждое изменение сигналов меняется значение счётчика.

``` if ( state == 4 ) { total_state++; state=0; } if ( state == -4 ) { total_state--; state=0; } ``` Зачем вот это я не понимаю, это не нужно, у вас есть переменная state, которая меняется каждый раз при изменении значений контактов. В остальном верно. Читайте теорию энкодеров, учите. В большинстве современных контроллерах блок, обрабатывающий сигналы с инкрементального энекодера реализованы аппаратно в таймере или в отдельном блоке, и на каждое изменение сигналов меняется значение счётчика.
stud126169 commented 2024-03-06 20:25:51 +00:00
Owner
Copy Link
      if ( state ==  4 )
      { 
        total_state++;
        state=0;
      }
      if ( state == -4 )
      { 
        total_state--;
        state=0;
      }

Зачем вот это я не понимаю, это не нужно, у вас есть переменная state, которая меняется каждый раз при изменении значений контактов. В остальном верно. Читайте теорию энкодеров, учите. В большинстве современных контроллерах блок, обрабатывающий сигналы с инкрементального энекодера реализованы аппаратно в таймере или в отдельном блоке, и на каждое изменение сигналов меняется значение счётчика.

Понял Степан Андреевич, подробнее прочитаю об этом, спасибо большое за помощь, а по нынешней версии кода, я реализовал полный цикл ктр проходит сигнал при повороте энкодера по часовой стрелке либо против часовой, и в зависимости от полученной комбинации мы получаем state равное 4 или -4 и в последствии от этого значения мы увеличиваем или уменьшаем на 1 наш результат

> ``` > if ( state == 4 ) > { > total_state++; > state=0; > } > if ( state == -4 ) > { > total_state--; > state=0; > } > ``` > Зачем вот это я не понимаю, это не нужно, у вас есть переменная state, которая меняется каждый раз при изменении значений контактов. В остальном верно. Читайте теорию энкодеров, учите. В большинстве современных контроллерах блок, обрабатывающий сигналы с инкрементального энекодера реализованы аппаратно в таймере или в отдельном блоке, и на каждое изменение сигналов меняется значение счётчика. Понял Степан Андреевич, подробнее прочитаю об этом, спасибо большое за помощь, а по нынешней версии кода, я реализовал полный цикл ктр проходит сигнал при повороте энкодера по часовой стрелке либо против часовой, и в зависимости от полученной комбинации мы получаем state равное 4 или -4 и в последствии от этого значения мы увеличиваем или уменьшаем на 1 наш результат
usr22772 commented 2024-03-13 15:44:23 +00:00
Owner
Copy Link

Хорошо Степан Андреевич, я еще раз прогоню все значения кода через debug и составлю таблицу со значениями и приложу ее сюда

Всё ещё жду

> Хорошо Степан Андреевич, я еще раз прогоню все значения кода через debug и составлю таблицу со значениями и приложу ее сюда Всё ещё жду
stud126169 commented 2024-03-13 16:47:08 +00:00
Owner
Copy Link

Вот составленная таблица, по нынешнему коду, не могли бы вы сказать то ли я составил или нет, заранее спасибо за вашу помощь

Вот составленная таблица, по нынешнему коду, не могли бы вы сказать то ли я составил или нет, заранее спасибо за вашу помощь
Таблица истинности энкодера (1).xlsx
11 KiB
stud126169 commented 2024-03-14 12:16:02 +00:00
Owner
Copy Link

Вот таблицы по старому коду и новому коду:

Вот таблицы по старому коду и новому коду:
Таблица истинности энкодера (1) (1).xlsx
11 KiB
stud126169 commented 2024-03-15 07:42:31 +00:00
Owner
Copy Link

Степан Андреевич, сбросил табличку для всех значений энкодера по старому коду

Степан Андреевич, сбросил табличку для всех значений энкодера по старому коду
Таблица истинности энкодера (1) (1) (1).xlsx
11 KiB
stud126169 commented 2024-03-15 08:03:35 +00:00
Owner
Copy Link

Степан Андреевич, сбросил табличку для всех значений по нынешнему коду

Степан Андреевич, сбросил табличку для всех значений по нынешнему коду
Таблица истинности энкодера (1) (1) (2).xlsx
11 KiB
usr22772 commented 2024-03-18 11:22:01 +00:00
Owner
Copy Link

всю дорогу прошу вот такую заполненную табличку image

всю дорогу прошу вот такую заполненную табличку ![image](/attachments/6f2fc5f5-b109-4fe0-b7f5-4881a9be3324)
image.png
10 KiB
stud126169 commented 2024-03-18 14:01:07 +00:00
Owner
Copy Link

Степан Андреевич, добрый вечер, переделал табличку как вы показали, прошу прощения что не смог понять что вы хотели, в самой таблице пустые столбцы означают что мы туда не попадаем по коду, соответственно значений с этих комбинаций нету, сама таблица сделана по старому коду

Степан Андреевич, добрый вечер, переделал табличку как вы показали, прошу прощения что не смог понять что вы хотели, в самой таблице пустые столбцы означают что мы туда не попадаем по коду, соответственно значений с этих комбинаций нету, сама таблица сделана по старому коду
Таблица истинности энкодера (1) (1) (1) (1).xlsx
11 KiB
stud126169 commented 2024-03-18 14:17:52 +00:00
Owner
Copy Link

Степан Андреевич, вот табличка по нынешнему коду

Степан Андреевич, вот табличка по нынешнему коду
Таблица истинности энкодера (1) (1) (1) (2).xlsx
11 KiB
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ICS-Microprocessor-2020/4_Ust-vo_poluchenia_velichin_s_datchikov#3
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?