Skip to content

location: update all location docs to reflect new structure#467

Merged
hasheddan merged 1 commit into
mainfrom
location/ga-updates
Jul 23, 2025
Merged

location: update all location docs to reflect new structure#467
hasheddan merged 1 commit into
mainfrom
location/ga-updates

Conversation

@hasheddan

Copy link
Copy Markdown
Contributor

Updates all Golioth Location documentation to reflect its new structure ahead of General Availability. Devices now interact with Golioth Location via Pipelines, creating a much more flexible and robust solution.

@github-actions

github-actions Bot commented Jul 23, 2025

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit a7222bb):

https://golioth-docs-dev--pr467-location-ga-updates-22kpdokk.web.app

(expires Fri, 22 Aug 2025 17:17:01 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f53b02bdc98ce6f5593931ec4c339aa96bac84df

@szczys szczys left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, nice work!

I left 3 comments that you can take or leave.

[Golioth Pipelines](/data-routing). For example, if tracking a device's location
over time is necessary, routing location to [LightDB
Stream](/application-services/lightdb-stream) or an external timeseries database
may be required. If it is necessary to make device's aware of their own

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
may be required. If it is necessary to make device's aware of their own
may be required. If it is necessary to make devices aware of their own

Golioth Location also offers built-in storage for the most recent device
position. Routing location information to the [`location` data
destination](/data-routing/destinations/location) will make it available for
visualization in `Location` section of the [Golioth

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
visualization in `Location` section of the [Golioth
visualization in the `Location` section of the [Golioth

Comment on lines +25 to +57
| Attribute | Description | Required |
|------------|------------------------------------------------------------------------------------|----------|
| `type` | String indicating type of cellular network (`ltecatm` or `nbiot` | X |
| `mcc` | Integer indicating Mobile Country Code. | X |
| `mnc` | Integer indicating Mobile Network Code. | X |
| `id` | Integer indicating EUCID (LTE Cat-M) or Cell ID (NB-Iot). | X |
| `strength` | Integer indicating RSRP (LTE Cat-M) or NRSRP (NB-Iot). (dBm) | |
| `lac` | Integer indicating Tracking Area Code (TAC) | |
| `age` | Integer indicating age of measurement. (ms) | |
| `channel` | Integer indicating EARFCN. | |
| `serving` | Boolean indicate whether device is currently served by the cell. (default: `false`) | |
| `lid` | Integer indicating PCI (LTE Cat-M) or NCID (NB-Iot) | |

#### Wi-Fi Access Point Format

| Attribute | Description | Required |
|-------------|--------------------------------------------------------------------------------------------------|----------|
| `mac` | String indicating access point MAC address. | X |
| `ssid` | String indicating access point Service Set Identifier. | |
| `strength` | Integer indicating signal strength of access point (dBm). | |
| `age` | Integer indicating age of measurement. (ms) | |
| `frequency` | Integer indicating frequency of access point. (MHz) | |
| `channel` | Integer indicating channel number of access point. | |
| `Connected` | Boolean indicating whether device is currently connected to the access point. (default: `false`) | |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to add any info about what fields are required and what are optional? I ask because the example do not include all fields.

I'm also wondering if there should be more clarification about age. Is this ms between the reading and submitting them to Golioth or is this an epoch?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to add any info about what fields are required and what are optional?

@szczys the last column indicates whether the field is required or not -- are you thinking we could restructure to make that more clear?

You also bring up a good point that it is probably worth at least adding a note that net_info support in the SDK can craft this for you.

I'm also wondering if there should be more clarification about age. Is this ms between the reading and submitting them to Golioth or is this an epoch?

age is the relative age of the measurement, so it would be more akin to the former. I can add a note to clarify 👍🏻

@dylanswartz dylanswartz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I appreciate the linking from the transformer to the application services section - i think it gives a good mental picture of how things fit together.

Updates all Golioth Location documentation to reflect its new structure ahead of
General Availability. Devices now interact with Golioth Location via Pipelines,
creating a much more flexible and robust solution.

Signed-off-by: Daniel Mangum <georgedanielmangum@gmail.com>
@hasheddan hasheddan force-pushed the location/ga-updates branch from 294a10c to a7222bb Compare July 23, 2025 17:13
@hasheddan hasheddan requested a review from szczys July 23, 2025 17:14

@szczys szczys left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for the additional context on required params and age!

@hasheddan hasheddan merged commit 54b91d5 into main Jul 23, 2025
2 checks passed
@hasheddan hasheddan deleted the location/ga-updates branch July 23, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants