⚙ [TO DO] Co chcielibyście jeszcze dodać do GUI-Generic? ☕

User avatar
YoMan
Posts: 2357
Joined: Thu Apr 30, 2020 5:18 pm
Location: Częstochowa

Post

SmartBOB wrote: Mon Apr 22, 2024 10:01 am
Jak coś to mogę wsprzeć autora zmian jakby udało się wykonać któryś z punktów ;)
To pisz na priva bezpośrednio :)
YoMan
________________________________________
Wziąłem udział w Supla Offline Party 2023 ;)
User avatar
uchnast.michal
Posts: 1102
Joined: Tue Jun 04, 2019 11:37 am

Post

YoMan wrote: Fri May 17, 2024 8:56 pm To pisz na priva bezpośrednio :)
A da się na priva nie bezpośrednio? Taki wpis troszkę na podobieństwo Twoich @YoMan ;)
e-talking............
Byłem tam - Supla Offline Party 2023
User avatar
YoMan
Posts: 2357
Joined: Thu Apr 30, 2020 5:18 pm
Location: Częstochowa

Post

uchnast.michal wrote: Tue May 21, 2024 5:27 pm
YoMan wrote: Fri May 17, 2024 8:56 pm To pisz na priva bezpośrednio :)
A da się na priva nie bezpośrednio? Taki wpis troszkę na podobieństwo Twoich @YoMan ;)
Taki zabieg liryczny mający na celu wzmocnienie przekazu 😉
YoMan
________________________________________
Wziąłem udział w Supla Offline Party 2023 ;)
User avatar
uchnast.michal
Posts: 1102
Joined: Tue Jun 04, 2019 11:37 am

Post

YoMan wrote: Tue May 21, 2024 5:57 pm
uchnast.michal wrote: Tue May 21, 2024 5:27 pm
YoMan wrote: Fri May 17, 2024 8:56 pm To pisz na priva bezpośrednio :)
A da się na priva nie bezpośrednio? Taki wpis troszkę na podobieństwo Twoich @YoMan ;)
Taki zabieg liryczny mający na celu wzmocnienie przekazu 😉
Waszeć argumenty uznaję;)
e-talking............
Byłem tam - Supla Offline Party 2023
SmartBOB
Posts: 61
Joined: Mon May 24, 2021 7:36 am
Location: Wrocław

Post

Pisałem na priva bezpośrednio też, @Krycha88 jest bardzo pomocny, udało się zrobić jednego szybkiego FIXa na bardzo dużego BUGa oraz poprawić obsługę LAN7820.
User avatar
YoMan
Posts: 2357
Joined: Thu Apr 30, 2020 5:18 pm
Location: Częstochowa

Post

SmartBOB wrote: Wed May 22, 2024 5:27 am Pisałem na priva bezpośrednio też, @Krycha88 jest bardzo pomocny, udało się zrobić jednego szybkiego FIXa na bardzo dużego BUGa oraz poprawić obsługę LAN7820.

Widzisz @uchnast.michal ? Jednak połączenie "priv" i "bezpośrednio" ma sens :lol: :lol: :lol:
YoMan
________________________________________
Wziąłem udział w Supla Offline Party 2023 ;)
rafalekkalwak@wp.pl
Posts: 527
Joined: Mon Feb 06, 2023 8:56 am

Post

Wsparcie dla SPS30 particulate matter sensor

https://github.com/krycha88/GUI-Generic/pull/133

@krycha88 jak możesz to rób squash merge z moich commitów bo zwykle robię ich wiele po trochu i ich nazwy pozostawiają wiele do życzenia :P

Mam nadzieję, że o niczym nie zapomniałem, a jak coś to pisz do dodam.
User avatar
klew
Posts: 8777
Joined: Thu Jun 27, 2019 12:16 pm
Location: Wrocław

Post

rafalekkalwak@wp.pl wrote: Sat May 25, 2024 6:12 pm Wsparcie dla SPS30 particulate matter sensor

https://github.com/krycha88/GUI-Generic/pull/133

@krycha88 jak możesz to rób squash merge z moich commitów bo zwykle robię ich wiele po trochu i ich nazwy pozostawiają wiele do życzenia :P

Mam nadzieję, że o niczym nie zapomniałem, a jak coś to pisz do dodam.
Nie wolno przypisywać autorstwa innym osobom :P. Dodajesz czujnik, to wpisz tam siebie :).
Poza tym ogólnie odradzam używanie namespace'ów "Supla" poza samą biblioteką. Wynika to z tego, że istnieje ryzyko, że w przyszłości dodam jakieś klasy (w namespace Supla) i przypadkiem nazwy będą te same i GG przestnie się kompilować. Co prawda łatwe do poprawienia, ale po co?
Widzimy się na Supla Offline Party vol. 2 :!:
rafalekkalwak@wp.pl
Posts: 527
Joined: Mon Feb 06, 2023 8:56 am

Post

klew wrote: Sat May 25, 2024 8:26 pm
rafalekkalwak@wp.pl wrote: Sat May 25, 2024 6:12 pm Wsparcie dla SPS30 particulate matter sensor

https://github.com/krycha88/GUI-Generic/pull/133

@krycha88 jak możesz to rób squash merge z moich commitów bo zwykle robię ich wiele po trochu i ich nazwy pozostawiają wiele do życzenia :P

Mam nadzieję, że o niczym nie zapomniałem, a jak coś to pisz do dodam.
Nie wolno przypisywać autorstwa innym osobom :P. Dodajesz czujnik, to wpisz tam siebie :).
Poza tym ogólnie odradzam używanie namespace'ów "Supla" poza samą biblioteką. Wynika to z tego, że istnieje ryzyko, że w przyszłości dodam jakieś klasy (w namespace Supla) i przypadkiem nazwy będą te same i GG przestnie się kompilować. Co prawda łatwe do poprawienia, ale po co?
Ale squash merge tylko zamiast kilku commitow zrobi jeden ,nadal jako ja?

Wiem że odradzasz a jednak wszystko w GG tak jest zrobione 🤫 no i o ile dobrze pamiętam to do supla device nie dorzucisz obcych bibliotek więc jeśli rozmawiamy o czujnikach to się pewnie nigdy nie wydarzy 😉
User avatar
klew
Posts: 8777
Joined: Thu Jun 27, 2019 12:16 pm
Location: Wrocław

Post

rafalekkalwak@wp.pl wrote: Sun May 26, 2024 5:38 am
Ale squash merge tylko zamiast kilku commitow zrobi jeden ,nadal jako ja?

Wiem że odradzasz a jednak wszystko w GG tak jest zrobione 🤫 no i o ile dobrze pamiętam to do supla device nie dorzucisz obcych bibliotek więc jeśli rozmawiamy o czujnikach to się pewnie nigdy nie wydarzy 😉
Chodziło mi o podpis w pliku.
Do sd mógłbyś dodać ten plik do obsługi czujnika, ale nie widzę potrzeby dodawania samych bibliotek. Raczej robimy to tak, że jest dodany komentarz, jakie biblioteki trzeba sobie doinstalować, aby dało się to skompilować.
GG ma wszystkie zależności też dodane do repo. Ogólnie można to zrobić inaczej, aby git te rzeczy pobierał
Widzimy się na Supla Offline Party vol. 2 :!:

Return to “GUI Generic”