mirror of
https://github.com/4IceG/luci-app-3ginfo-lite.git
synced 2026-04-25 11:25:50 +03:00
[GH-ISSUE #47] 3ginfo.sh working from shell but not populating luci #167
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/luci-app-3ginfo-lite#167
Loading…
Add table
Add a link
Reference in a new issue
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?
Originally created by @brian4286 on GitHub (Sep 5, 2023).
Original GitHub issue: https://github.com/4IceG/luci-app-3ginfo-lite/issues/47
Running 1.0.54-20230903 I purged all and forced a reinstall via opkg.
Seems like the output from the shell script is working but I am not seeing it render in LUCI. I am testing in OpenWrt 23.05.0-rc3 r23389-5deed175a5 so this could be related, however it was working through rc2.
Happy to dig deeper, wanted to validate I am not alone before I do.
@4IceG commented on GitHub (Sep 5, 2023):
Hi @brian4286 , it seems that your sim card is not logged into the network, and this is required for the LuCI package.
@4IceG commented on GitHub (Sep 5, 2023):
Show result of the command:
sh -x /usr/share/3ginfo-lite/3ginfo.sh
@brian4286 commented on GitHub (Sep 5, 2023):
@4IceG sim is working fine, the modem is online and the AT commands are working as expected. I am sorry I did not follow up sooner but I ran debug earlier the script, it is working. I was trying to find why the variables were not being displayed. I can try to modify the script and echo the variables to figure out which one errors.
I will try to get to that later and debug more from my side as well, apologies dumping it and not providing this output.
@4IceG commented on GitHub (Sep 5, 2023):
@obsy Cezary chyba znow będą skrypty do korekty.. ciagle coś wychodzi :)
@brian4286 commented on GitHub (Sep 5, 2023):
For example, where the variable for $CSQ is created, I immediately echo'ed out the variable and it works. So somewhere below that the variable gets cleared.
@obsy commented on GitHub (Sep 5, 2023):
To jest tryb połączenia którego u nas nie ma i nie parsujemy: 11, czyli wg dokumentacji "NR connected to 5GCN". Wg danych wygląda jak czyste 5G (SA). Zdecydowanie do dodania do skryptów, bo teraz po prostu tego nie analizujemy.
@4IceG commented on GitHub (Sep 5, 2023):
Widze że u Nas tego nie ma, widać to też po pasmach CA.
@obsy commented on GitHub (Sep 5, 2023):
@brian4286: this is a new mode that requires our script changes to be supported. Right now the scripts don't parse it like they should.
@brian4286 commented on GitHub (Sep 5, 2023):
5GSA FTW? 😄
@obsy commented on GitHub (Sep 5, 2023):
@4IceG: tylko że wg tych danych +CREG: 2,0 czyli wg dokumentacji on nie jest zarejestrowany w sieci i nie szuka sieci. Czyli wg ideologi dane nie powinny być parsowane bo aktywnego połączenia nie ma. Jeżeli on ma połączenie jak napisał to znów coś quectel kłamie i nie mamy statusu rejestracji karty sim w sieci.
@4IceG commented on GitHub (Sep 5, 2023):
@obsy Pewnie zaraz wyjdzie że jest jakies polecenie +CREG5G:, ktore dane pokazuje.. i tyle po Naszej logice. Bądź tu mądry.. tylko na koncu czego się trzymać i czym kierowac.
@obsy commented on GitHub (Sep 5, 2023):
@brian4286: Do you have a normal, working internet connection? Can you show the result again:
after ifdown wan; ifup wan (or change "wan" if you have a section with a name other than wan).
@obsy commented on GitHub (Sep 5, 2023):
AT+C5GREG jest :)
@brian4286 commented on GitHub (Sep 5, 2023):
Here is the debug output from the shell script:
Yes I am online:
I don't call it wan, it is called wwan on my box if that helps.
@obsy commented on GitHub (Sep 5, 2023):
And show
sms_tool -d /dev/ttyUSB3 at "AT+C5GREG?"
@brian4286 commented on GitHub (Sep 5, 2023):
Here you go...
@obsy commented on GitHub (Sep 5, 2023):
Better:
sms_tool -d /dev/ttyUSB3 at "AT+C5GREG=2;+C5GREG?"
??
@brian4286 commented on GitHub (Sep 5, 2023):
@obsy commented on GitHub (Sep 5, 2023):
Thanks. Ok, we should be able to do something with that data.
@brian4286 commented on GitHub (Sep 5, 2023):
Just @ me and I will be happy to pull down the change and test before you package it up.
@obsy commented on GitHub (Sep 6, 2023):
First try:
@4IceG : o ile pamiętasz to nie tak dawno walczyliśmy z parametrami dla SCC, bo się okazało że dla naszego NR5G-NSA inaczej trzeba to wyliczać (brak było pola "state"). No to tu dla utrudnienia nie ma pola "state" dla PCC więc wylicza się to inaczej, ale jest "state" dla SCC i bandu NR5G, więc znów wylicza się inaczej. Robi się cała masa if'ów.
No i nie wiem czy 0 dla RSRP traktować jako brak parametru (tak samo jak "-") czy to jest faktyczna wartość tylko chwilowo ma właśnie 0.
@4IceG commented on GitHub (Sep 6, 2023):
@obsy U Ciebie Cezary wyglada fajnie, mam nadzieje że jakoś odnajde się w tych skryptach i uda się to przenieść dla LuCI.
@obsy commented on GitHub (Sep 8, 2023):
@brian4286 : work in progress
@4IceG commented on GitHub (Sep 10, 2023):
Hi @brian4286 ,
Check the new version of the package.
@brian4286 commented on GitHub (Sep 11, 2023):
@4IceG I am on a different tower right now so can't be certain if it is parsing correctly. I will update this issue when I am back at that tower over the next week or two.