#21027 closed enhancement (fixed)
Color Mapillary tracks by capture date
Reported by: | pyrog | Owned by: | taylor.smock |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Plugin mapillary | Version: | |
Keywords: | Cc: |
Description
Attachments (1)
Change History (7)
by , 4 years ago
Attachment: | Color images by capture date.png added |
---|
follow-up: 2 comment:1 by , 4 years ago
follow-up: 4 comment:2 by , 4 years ago
Replying to taylor.smock:
I'm assuming something like the website?
yes
This will be a bit more long-term -- I want to get things "stable" or at least "stablish" first.
Of course. Take time to get a stable version 😀
comment:3 by , 4 years ago
I can think of lots of different coloring settings similar to GPS tracks. Maybe some core code can be reused.
comment:4 by , 4 years ago
Replying to pyrog:
Of course. Take time to get a stable version 😀
I'm hoping to start writing actual tests this week :) . Instead of writing hotfixes :( .
Replying to skyper:
I can think of lots of different coloring settings similar to GPS tracks. Maybe some core code can be reused.
Maybe. I'd have to look -- I'm using the newish vector tile code (except for the drawing), and I don't know how the GPX tracks are getting drawn.
comment:5 by , 4 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I'm going to mark this as fixed -- see b7d1c09cc88b8f27669d275c62efed2457dd3827.
I looked into seeing if I can reuse the GPS colouring options, but this was quick and matched the option on the Mapillary website.
I'd have to do a fair amount of JOSM core work to be able to reuse the GPS drawing information (it uses GpxDrawHelper, which only allows GpxLayer's). I would then need to override createMapViewPainter
, but that would be the easy part. :)
comment:6 by , 4 years ago
Actually, I was thinking of more options than just date but also author and other information but I do not really care.
I'm assuming something like the website? This will be a bit more long-term -- I want to get things "stable" or at least "stablish" first.