mnalis left a comment (openstreetmap/openstreetmap-website#5904)

**TL;DR:** great for adding support for adding/changing (optional) tags on 
notes, as well as (optional) extra API params / website changes to filter by 
them; but please no mutable note texts/coordinates - it would be quite 
problematic!

---

Thanks for clarifications @nenad-vujicic !

> -  At the moment mappers use hash-tags. In near future we could use buttons 
> for creating / updating / deleting note tags. Similarly for reading, instead 
> of passing through note-comments, we will have a table like for elements 
> (this is implemented).

I agree; adding tags to notes is useful feature. Data consumers which do not 
implement that extension would still be able to use all existing functionality 
in the same way, so would only miss on those extensions.

> -   At the moment mappers manually search / filter notes. In near future we 
> could use UI for defining searching / filtering criterias.

Having an API extension being able to also filter by those tags would be nice 
addition, getting around limitation of maxnotes and local filtering etc. 
(provided it reduces the server load instead of increasing it). Also, data 
consumers which do not implement those search extensions would retain same 
functionality as before, so this sounds good to me too.

> -   At the moment mappers manually inspect notes or use 3rd party tools. We 
> already have implemented (in this PR) retrieving particular note's version, 
> which gives us a chance to build note's history viewers / analyzers for 
> automatizing parts of note's inspection.

**Mutable text/coordinates of Notes however looks highly problematic to me.** 

Notes are currently journal-like implementation, i.e. only new comments can be 
added, and old ones cannot be changed. I think making the comments and 
coordinates mutable would be very bad. It is IMHO **not comparable to elements 
(nodes/ways/relations) versioning**, as those are self-contained and **do not 
have dependent subelements**. But notes have: each note comment is dependent 
"subelement", i.e. meaning of each note "reply" **depends** on the content of 
**coordinates and original texts of ALL preceding comments**, and **not** on 
their latest version! 

Changing the text or coordinates of an original note (or previous comment) at a 
later time, (potentially) changes the meaning of all comments that were written 
between time where original text was written and its updated version. Also, 
some apps cache notes offline, which would further make determining on which 
version the reply was made problematic. Also, one should consider that there 
will be both old API apps and new API apps running at the same time for 
forseeable future, so they would need to work together without stepping on each 
other toes.

I.e. in current Notes system it is extremely easy and efficient to follow the 
temporal ordering of events, just follow the "comments" by increasing 
ids/timestamps. Any clarifications or corrections are just another comment down 
the line. 

Making the comments text or coordinates mutable would totally break the 
workflow of all existing note consumers, and require all consumerts to convert 
to new API _and_ also invest significant effort to make sense of and avoid 
misinterpreting Notes, for no good reason. 

Also, how would the [Notes planet 
dump](https://planet.openstreetmap.org/notes/) change with mutable notes? It 
seems like a drastic change, perhaps even necessitating splitting to separate 
planet-notes and history-notes as we have for elements  (compared to e.g. mere 
adding of note  tags, which would probably just add "tags=" attribute to 
appropriate  XML element, which would work as-is in most if not all data 
consumers, and is trivial change)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5904#issuecomment-2877798403
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/5904/c2877798...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to