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

nodeDB only seem to be updated when the sending device gets a gps location lock #679

Closed
mc-hamster opened this issue Feb 8, 2021 · 10 comments
Assignees
Labels
bug Something isn't working

Comments

@mc-hamster
Copy link
Member

The values of n where ( NodeInfo *n = nodeDB.getNode(mp.from); ) only seem to be updated when the sending device gets a gps location lock (and maybe from a phone, I haven’t looked). Given that, n->user.long_name and n->user.short_name are both blank until the sender gets a location lock. In another round of tests, n->user.long_name and n->user.short_name are both unset for half an hour even after the location is updated in the receiver’s node database. Is this expected behavior? Is there a bug here?

Edit: Sometime between 20 and 60 minutes in (I my terminal ran out of buffer) n->user.long_name and n->user.short_name got populated. Regardless, that’s a long time for a new node to know who else is on the mesh.

@geeksville
Copy link
Member

This issue has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/on-device-api-now-available-feedback-requested/1872/28

@mc-hamster
Copy link
Member Author

@geeksville geeksville added the bug Something isn't working label Feb 8, 2021
@geeksville
Copy link
Member

great find! i'll investigate tonight.

@mc-hamster
Copy link
Member Author

I should add ...

The sender was configured to send a packet once every 60 seconds, so it's not as though it didn't have an opportunity to send something over the air.

@geeksville geeksville assigned mc-hamster and unassigned geeksville Feb 10, 2021
@geeksville
Copy link
Member

hi @mc-hamster
re: name updates being slow
hmm - I'm having a hard time reproducing this if send_owner_interval is the default of 4 and position_broadcast_secs is 60, it should send the owner info every 240 seconds. Is that not happening?

@mc-hamster
Copy link
Member Author

I was working with device defaults. I'll get you better reproduction steps this weekend.

@mc-hamster
Copy link
Member Author

Take a look at the attached CSV.

At timestamp 00:44:04 both sender and receiver were restarted.

From timestamp 00:44:00 -> 01:18:57 the receiver never got the sender's name.

At 1:20:52, the sender restarted and then the name started to show up.

rangetest-26.csv.zip

@mc-hamster
Copy link
Member Author

I'm still seeing this but will retest after 1.2 comes out. Maybe it'll be magically fixed by then.

@sachaw
Copy link
Member

sachaw commented Dec 12, 2021

@mc-hamster can you please confirm that this is now fixed?

@mc-hamster
Copy link
Member Author

I have not seen this issue in 1.2.

Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants