Dronekit delivery tracking needs to zoom and also ideally auto update #537

Open
hamishwillee opened this Issue Feb 8, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@hamishwillee
Contributor

hamishwillee commented Feb 8, 2016

The track screen on the dronekit delivery example should ideally be updated to:

  • Track the position of the vehicle without having to push the update button
  • Support zoom
  • Zoom initially to a reasonable level based on the distance between the initial position and the target position.
@kosovit

This comment has been minimized.

Show comment
Hide comment
@kosovit

kosovit Jul 25, 2016

Hi @hamishwillee !
to make an autoupdate add change in track.html the following :

`

    {% include 'head.html' %}
`

to

`

    {% include 'head.html' %}
     <meta http-equiv="refresh" content="1"> <!-- See the difference? -->
</head>`

This will make an auto refresh !

To make it support the zoom, I think there is a big work to do but you can modify the option zoom in the python file: change 'zoom': 13, to 'zoom': 19, . 19 is the maximun !

finally I have a question
since a week I did not see the map in the command and in the track . I think that there is a problem with
access_token': 'pk.eyJ1Ijoia2V2aW4zZHIiLCJhIjoiY2lrOGoxN2s2MDJzYnR6a3drbTYwdGxmMiJ9.bv5u7QgmcJd6dZfLDGoykw',
'mapid': 'kevin3dr.n56ffjoo',
I think I should change it ,
do you have the same issue ?
did you try to figure out ?

kosovit commented Jul 25, 2016

Hi @hamishwillee !
to make an autoupdate add change in track.html the following :

`

    {% include 'head.html' %}
`

to

`

    {% include 'head.html' %}
     <meta http-equiv="refresh" content="1"> <!-- See the difference? -->
</head>`

This will make an auto refresh !

To make it support the zoom, I think there is a big work to do but you can modify the option zoom in the python file: change 'zoom': 13, to 'zoom': 19, . 19 is the maximun !

finally I have a question
since a week I did not see the map in the command and in the track . I think that there is a problem with
access_token': 'pk.eyJ1Ijoia2V2aW4zZHIiLCJhIjoiY2lrOGoxN2s2MDJzYnR6a3drbTYwdGxmMiJ9.bv5u7QgmcJd6dZfLDGoykw',
'mapid': 'kevin3dr.n56ffjoo',
I think I should change it ,
do you have the same issue ?
did you try to figure out ?

@hamishwillee

This comment has been minimized.

Show comment
Hide comment
@hamishwillee

hamishwillee Jul 25, 2016

Contributor

Hi @kosovit

Those changes sound pretty reasonable to me, though we would also need to tidy up the documentation.

How about you create a patch that we can review?

Contributor

hamishwillee commented Jul 25, 2016

Hi @kosovit

Those changes sound pretty reasonable to me, though we would also need to tidy up the documentation.

How about you create a patch that we can review?

@munkarkin96

This comment has been minimized.

Show comment
Hide comment
@munkarkin96

munkarkin96 Jan 28, 2017

Hi @hamishwillee ,

If you don't mind me asking, how's the progress of this issue going?

Thanks!

Hi @hamishwillee ,

If you don't mind me asking, how's the progress of this issue going?

Thanks!

@hamishwillee

This comment has been minimized.

Show comment
Hide comment
@hamishwillee

hamishwillee Jan 30, 2017

Contributor

@munkarkin96 Not at all. This is more of an "enhancement", that we were hoping others would do.

Contributor

hamishwillee commented Jan 30, 2017

@munkarkin96 Not at all. This is more of an "enhancement", that we were hoping others would do.

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