dl9rdz / rdzwx-go

Apache License 2.0
22 stars 3 forks source link

Prediction does not work #5

Open WH3080 opened 3 years ago

WH3080 commented 3 years ago

Hi, I really love both the constantly improving firmware and now also the app. I actually didn't realize I could download apk files so I installed the complete tool chain on a windows 10 machine and compiled my own apks. The debug version runs on a https://www.samsung.com/us/business/products/mobile/tablets/galaxy-tab-active/explore/, but not the release version. Also, I noted that the prediction is not working. It always causes a "bad request error". Yesterday I realized that I can download compiled versions from GitHub. Tried to install one, but it failed just like the release version of my self compiled apk. I noticed that the ready to install apk is > 2MB in size while the self compiled debug version is around 1MB. Any thoughts how I can get the GitHub ready apk version to work including the prediction feature? Thank you!

dl9rdz commented 3 years ago

The prediction should work, it works for me and several other people. However, it requires internet access. It uses the CUSF API, if that is down, it will fail.

Maybe you connected your phone to the TTGO (with TTGO in access point mode and Phone in WiFi client mode)? In that case, the phone will not have internet access (unless you do some magic with third party tools that enable Wifi client for 192.168.4.0/24 and Internet access via mobile data. The easier option is to have your phone in WiFi access point mode and let your TTGO connect to your phone.

Of course you need to have mobile data enabled on your phone and be in a place with network coverage.

dl9rdz commented 3 years ago

Regarding the Samsung, do you mean the release version compiled on your own toolchain? That would be strange. As you have the toolchain installed, maybe you can run "adb logcat" on command line or view logcat in Android Studio and filter the output for dl9rdz and sent that to me?

(The apk on git releases does not contain the latest bug fixes in the repository, I am too busy at the moment, probably I will create an updated version in two weeks. There is a known bug that causes a crash if GPS is not available. So if you mean the apk on git with "release version", this might be the reason)

dl9rdz commented 3 years ago

Final comment regarding the file size of the APK:

For converting ellipsoid height (from sonde) into sea level height, the APK should contain the world wide 15' data base (in assets/WW15MGH.DAC), which is about 2MB uncompressed, 1.7 MB compressed.

Oh, just checked, my fault. I did not push that part to git yet (in rdzwx-plugin). Well, I just did now, but not guarantee that it is functional. I had some test code in it that should automatically generate bug reports upon crashes by email, but that was working only with some, but not all mail apps. I also have some test code in it for some standalone predictor (that works without Internet access), but this is not yet ready for release and should be present but deactivated in the code.

WH3080 commented 3 years ago

The prediction should work, it works for me and several other people. However, it requires internet access. It uses the CUSF API, if that is down, it will fail.

Maybe you connected your phone to the TTGO (with TTGO in access point mode and Phone in WiFi client mode)? In that case, the phone will not have internet access (unless you do some magic with third party tools that enable Wifi client for 192.168.4.0/24 and Internet access via mobile data. The easier option is to have your phone in WiFi access point mode and let your TTGO connect to your phone.

Of course you need to have mobile data enabled on your phone and be in a place with network coverage.

a) I did not mean to imply that it doesn't work in general. In fact I admire your work no kidding. b) The Samsung Active 3 is a tablet, not a phone. c) I tested it at home with WiFi. d) I can easily access the internet, i.e. from your TTGO webpage to link to Google maps. Both the Samsung tablet and TTGO are in the same home WiFi, one in the 2.4G and the other in 5G e) I don't know how to test if that API is down you mentioned.

WH3080 commented 3 years ago

Regarding the Samsung, do you mean the release version compiled on your own toolchain? That would be strange.

As you have the toolchain installed, maybe you can run "adb logcat" on command line or view logcat in Android Studio and filter the output for dl9rdz and sent that to me?

(The apk on git releases does not contain the latest bug fixes in the repository, I am too busy at the moment, probably I will create an updated version in two weeks. There is a known bug that causes a crash if GPS is not available. So if you mean the apk on git with "release version", this might be the reason)

a) I followed all the steps on your GitHub page on a Windows 10 machine. The compilation results are a debug and a release apk. b) I tried to install both - one at a time of course. Debug works except prediction. Release can not be installed for some odd reason. c) Only later I found out I could have avoided the whole toolchain ordeal and simply downloaded your release version from GitHub. However it won't install. Privileges and security should not be the issues as I "accepted the risk installing from an unknown source".

WH3080 commented 3 years ago

Final comment regarding the file size of the APK:

For converting ellipsoid height (from sonde) into sea level height, the APK should contain the world wide 15' data base (in assets/WW15MGH.DAC), which is about 2MB uncompressed, 1.7 MB compressed.

Oh, just checked, my fault.

I did not push that part to git yet (in rdzwx-plugin). Well, I just did now, but not guarantee that it is functional. I had some test code in it that should automatically generate bug reports upon crashes by email, but that was working only with some, but not all mail apps. I also have some test code in it for some standalone predictor (that works without Internet access), but this is not yet ready for release and should be present but deactivated in the code.

Good to know. Will try to compile again with new git pull. But not tonight. I have a day time job too 😉.