[PATCH][User-Manual][Companion-app]-Minor-changes-on-app-and-manual.patch

[Salvador Cuñat]

On Wed, Apr 23, 2014 at 07:36:00AM -0700, Dirk Hohndel wrote:
> On Tue, 2014-04-22 at 20:14 +0200, Salvador Cuñat wrote:
> > +- Maps:  Will display a map showing the dive locatión (you'll be prompted to
> 
> did you mean "location" ?
> 
Off course I did  :-)

> > +choose which auxiliar app use from your installed apps). At this date this feature is
> 
> auxiliary or maybe just "helper app to use"
> 
"Helper" might be more correct as they are full apps in their own right.

> > +a bit of a proof of concept, is expected to be fully functional by end of
> > +May/2014.
> 
> I wouldn't give a hard date like that :-)
> 
Well, Aurelien expected to have it by the end of April and I added a
month, just for ... BTW, as Venkatesh says, there are modifications on the
repository, *probably* functional but still untested in real diving,
just waiting to be in Playstore.  Off course the text can be changed
with a simple "soon".

> > +
> > +date-times of the uploaded GPS localities. If you had hand-writen the name of the dive
> 
> maybe "entered" instead of "hand-written"
>
Debatable. I used his term explicitly to remark the difference but
"entered" is perfect and surely more understandable.

> > +A dirty hack around this is manually editing the date-time of a dive in Subsurface's
> 
> You meant to say "A workaround for this situation is to manually
> edit..." :-)
>
I did, but it's still a dirty hack :-)

> >> > +TIPS:
> > +
> > +- _Background service_, being a very powerful tool, may fill your register in web service with
> > +lots of annoying fake coordinates. These coordinates are, today, dificult and bothering to
> > +delete from the server. Is strongly recomended you make a clean up in your android device
> > +before sending the dive points to the web server, by simply deleting the fake positions in the
> > +device.
> > +
> > +- Same way, I would recommend to give significant names to the dives sent to the web server, or
> > +, at least, use a significant name in the _Name Template_ setting while running _background service_,
> > +especially if you are in a dive trip and you are piling up lots of dives and dive points waiting to
> > +come back home to download them to _Subsurface_.
> 
> And the last section called TIPS... I simply disagree with what you
> write there. People who use the background service don't use the
> companion app the way you envision. I have several hundred records in my
> companion app because I frequently let it simply run all day. The
> coordinates aren't "fake" - they are real coordinates at certain
> time.
>
May be "useless" is the correct word as "fake" sounds too tough, but
they are no diving points yet, just travelling points. I'll change this
text to reflect that deleting them may be necesary *only* in some cases.

> There is no reason to trim the list and delete any of those. That's the
> whole point. Subsurface does the right thing and finds the matching
> location based on time stamps. I don't want to suggest to users that
> they need to do that manually - that takes the whole usefulness of the
> companion app in background mode away!

The way I use background service is, probably, too personal because of
some peculiarities around some of the dive points I usually dive. When I
use background service I end with 100/120 locations for a single dive,
being just one of them the correct one. This is why I made such a
recomendation. If I only had 3 or 4 would simply let it run.

Lastly, if the user has his server account filled with meaningless
coordinates, the map on the web page won't be of great use.

Attached is a modified patch with the issues commented here and a
modification I forgot to include in yesterday's one.

Please, take a look on it.

Regards.

Salva.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: [User-Manual][Companion-app]-Minor-changes-on-app-and-manual.patch
Type: text/x-diff
Size: 6863 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140423/02ba2909/attachment-0001.patch>