Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.1.11-alpha Bug Reports: Add your bugs here! #53

Closed
E3V3A opened this issue May 2, 2014 · 1 comment
Closed

v0.1.11-alpha Bug Reports: Add your bugs here! #53

E3V3A opened this issue May 2, 2014 · 1 comment

Comments

@E3V3A
Copy link
Contributor

E3V3A commented May 2, 2014

This version look very nice and most things just work. :)
Here are the very few things that doesn't quite work.

  • Database Viewer: "Cell Data" has entries with CellID = -1 (?)
  • Settings: Default icon-set not selected/filled. (Default: "sense")
  • View Map: This is weird. I guess we need to graphically distinguish between OpenCellID towers and those connected to by phone? (It freaks you out when you see two cell towers right on top your own GPS location, when you know there are none there.)

I have forgotten if the API give you the GPS of the cell towers or not? If they do, we need to separate those coordinates (somewhere) with those found in the OpenCellID DB, if they do not agree. I guess there will be 3 GPS positions in the AIMSICD DB:

  1. One for OCI GPS co's, and
  2. the other for Network/tower provided GPS and
  3. our own/phone GPS coordinates.

This was tested on:

  • GT-I9100, Custom 4.4.2
  • GT-I9195, Stock 4.2.2
@E3V3A E3V3A added bug labels May 2, 2014
@xLaMbChOpSx xLaMbChOpSx self-assigned this May 3, 2014
@xLaMbChOpSx
Copy link
Contributor

Latest release address the default icon issue finally I kept forgetting about that one.

The -1 for the CellID is strange as it should be assigned a value (-1 is the default initialisation value), I will have a look at this and worst case add an additional check prior to insert to ensure this can not happen.

The Cell marker issue you raised I believe is an error on my behalf as I created a marker within the section that attempts to display signal details so is creating a marker with the CellID whilst iterating through any signal strength database entries I will remove this in the next commit as obviously it is incorrect!! I will put that one down to maybe drinking bourbon while coding on a Friday night!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants