E.g. a particular restaurant in a franchise chain could be operated by some small/unknown company. See https://wiki.openstreetmap.org/wiki/Key:operator#Examples

Hmm, maybe there was a reason "operator" was picked up for certain POIs only - probably only for those where it should highly likely make sense. E.g. a particular restaurant in a franchise chain could be operated by some small/unknown company. See https://wiki.openstreetmap.org/wiki/Key:operator#Examples

vng commented

2024-06-29 18:56:27 +00:00

(Migrated from github.com)

It is a follow-up from here and here

You can open a discussion there :)

It is a follow-up from [here](https://git.omaps.dev/organicmaps/organicmaps/pulls/8523) and [here](https://git.omaps.dev/organicmaps/organicmaps/issues/8039) You can open a discussion there :)

pastk commented

2024-06-29 18:57:21 +00:00

Member

I did :)

vng commented

2024-06-29 19:02:13 +00:00

(Migrated from github.com)

I also have mixed feelings about widely used 'operator' and 'network' tags, but POIs look like a reasonable tradeoff.

biodranik commented

2024-06-29 19:39:54 +00:00

(Migrated from github.com)

What are the typical values in some well-mapped city? Let's see if they're useful for POIs.

vng commented

2024-06-29 22:07:18 +00:00

(Migrated from github.com)

Ok, while the discussion is still open, I made like before (operator selector) plus "university", "vending_machine" (from the issue).

pastk commented

2024-06-30 09:12:07 +00:00

Member

The issue mentioned public transport ticket vending machines - makes sense.

Re other vending machines (e.g. drinks, coffee) it could be some small unknown companies operating them... But IDK really.
It'd be good if someone makes a tagging research.
Or we can push it to alpha and ask for feedback.

The issue mentioned public transport ticket vending machines - makes sense. Re other vending machines (e.g. drinks, coffee) it could be some small unknown companies operating them... But IDK really. It'd be good if someone makes a tagging research. Or we can push it to alpha and ask for feedback.

pastk commented

2024-06-30 09:13:42 +00:00

Member

Review: Approved

Technically it looks good!

**Review:** Approved Technically it looks good!

pastk commented

2024-06-30 09:13:42 +00:00

Member

approved this merge request

pastk commented

2024-07-01 14:49:35 +00:00

Member

The issue mentioned public transport ticket vending machines - makes sense.

Re other vending machines (e.g. drinks, coffee) it could be some small unknown companies operating them...

Is it easy to limit the vending machines change to amenity-vending_machine-public_transport_tickets only?

> The issue mentioned public transport ticket vending machines - makes sense. > > Re other vending machines (e.g. drinks, coffee) it could be some small unknown companies operating them... Is it easy to limit the vending machines change to `amenity-vending_machine-public_transport_tickets` only?

dvdmrtnz commented

2024-07-01 19:11:07 +00:00

Contributor

Changes look good, only one comment about this:

Ok, while the discussion is still open, I made like before (operator selector) plus "university", "vending_machine" (from the issue).

In #8039 the user actually wanted to be able to see the operator of buildings operated by a university. So these buildings would not be tagged with amenity=university but with building.

I understand that most building's operators are going to be irrelevant for users, so maybe only show operator for university-related buildings like building=dormitory, building=university ?

Changes look good, only one comment about this: > Ok, while the discussion is still open, I made like before (operator selector) plus "university", "vending_machine" (from the issue). In https://git.omaps.dev/organicmaps/organicmaps/issues/8039 the user actually wanted to be able to see the operator of buildings operated by a university. So these buildings would not be tagged with `amenity=university` but with `building`. I understand that most building's operators are going to be irrelevant for users, so maybe only show operator for university-related buildings like `building=dormitory`, `building=university` ?

vng (Migrated from github.com) closed this pull request

2024-07-07 01:48:41 +00:00

vng (Migrated from github.com) merged commit into master

2024-07-07 01:48:41 +00:00

dvdmrtnz commented

2024-07-20 15:51:11 +00:00

Contributor

mentioned in issue #5219