WiFiAnalyzer icon indicating copy to clipboard operation
WiFiAnalyzer copied to clipboard

Implement GPS location, and bearing acquisition and export.

Open skinkie opened this issue 3 years ago • 7 comments

Three years and 10 days after the suggestion https://github.com/VREMSoftwareDevelopment/WiFiAnalyzer/issues/152#issuecomment-406803520 and being inspired by @mores, a fix for #152

What does this implement/fix? Please describe. Export longitude,latitude,accuracy,bearing,bearingaccuracy along with the WiFiDetails.

Does this close any currently open issues? #152

Additional context This feature would be really useful if the app would support continuous exports, or write something to a file.

I am really a novice programmer in the Kotlin language. It might be worthwhile to move the location activity to the MainContext, to make the variable private (again).

Where has this been tested?

  • Operating System: Linux
  • Platform: Android 10
  • Target Platform:
  • Toolchain Version:
  • SDK Version:

skinkie avatar Jul 31 '21 12:07 skinkie

@VREMSoftwareDevelopment any reason to close this issue without any feedback?

skinkie avatar Mar 05 '22 18:03 skinkie

@skinkie there were not response to the PR feedback provided by us for over 1/2 year. Which stand to show that there no interest fixing the issues in the PR. It is always possible to re-open existing PR or submit a new PR with new code changes.

VREMSoftwareDevelopment avatar Mar 05 '22 19:03 VREMSoftwareDevelopment

A non-project member is not able to open a closed pull request. So yes, an open source product could be forked, but still it does not make sense when a feature is provided no reply at all is provided. With respect to 'interest' as referenced, end-users have expressed interest, including myself.

skinkie avatar Mar 05 '22 19:03 skinkie

We have re-opened the PR, please response to our feedback(s).

VREMSoftwareDevelopment avatar Mar 05 '22 19:03 VREMSoftwareDevelopment

Please make sure that this PR solves concerns raised in PR #286, since they are related.

Thanks

VREMSoftwareDevelopment avatar Mar 05 '22 20:03 VREMSoftwareDevelopment

Thanks. Very strange I have not received a notification about this. I'll see if I can make it work with the provided feedback.

skinkie avatar Mar 05 '22 20:03 skinkie

Here a few links to reference documentation that may help: Optimize location for battery Background Location Limits

VREMSoftwareDevelopment avatar Mar 05 '22 20:03 VREMSoftwareDevelopment