Conversation
Terms were suggested by LLM and human-reviewed (around 90% of suggestions were discarded for various reasons). the main LLM command was: `add missing useful search terms very likely to be used by mappers to identify given POI when searching for presets in editors, using US English, if existing terms cover it do not add it. Especially if terms has multiple words and first of them is term already then value is really low (search terms are added in 'terms' property)`
|
@ZeLonewolf are you maybe interested in a review also of this one? |
| "terms": [ | ||
| "collateral", | ||
| "loan", | ||
| "secondhand" |
There was a problem hiding this comment.
A secondhand store seems very different from a pawnshop.
There was a problem hiding this comment.
maybe it is just local oddity, but in Poland pawnshops are also selling stuff left as a collateral
is it normal elsewhere?
There was a problem hiding this comment.
Yes, that's true. But there's a whole separate category of store called a secondhand shop. So if someone searches for secondhand, it should definitely put that higher in the list than pawnshop.
There was a problem hiding this comment.
and it works this way!
pawnshop get listed after thrift stores (shop=second_hand) and second-hand clothing stores, which is IMHO better than not being listed at all on such search term
| "name": "Pawnshop", | ||
| "terms": [ | ||
| "collateral", | ||
| "loan", |
There was a problem hiding this comment.
Here, loan stores are known as "payday lenders" and very different from pawn shops.
There was a problem hiding this comment.
my intention is that someone typing in loan may mean either of those so listing both makes sense
| }, | ||
| "name": "Pawnshop", | ||
| "terms": [ | ||
| "collateral", |
There was a problem hiding this comment.
Might apply but terminology not used in my country
There was a problem hiding this comment.
interesting, different US people just responded that it is used
(my question was
pawnshop questions...
Is word "collateral" used in USA?
Is typical pawnshop also selling stuff people loaned against and failed to recover?
to clarify - I am asking about collateral in pawnshop context, and not about unintentionally bombed pawnshops (as in "collateral damage")
)
I asked people on non-OSM group.
There was a problem hiding this comment.
I will leave it on account not being wrong and at least plausible to be used by non-natives and maybe used in USA.
| @@ -15,5 +15,13 @@ | |||
| "shop": "fuel" | |||
| }, | |||
| "name": "Fuel Shop", | |||
There was a problem hiding this comment.
Not familiar enough with this type of store to know if these items are typically sold there.
|
🍱 Your pull request preview is ready Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone. FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes. |
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
| "kerosene", | ||
| "pellets" | ||
| ], | ||
| "matchScore": 0.5 |
There was a problem hiding this comment.
Since this has a match score of 0.5, maybe adding more presets to make it easier to be found is actually counterproductive?
There was a problem hiding this comment.
I guess I can check why it got so low match score
But these terms are quite indicative it is this (what else should match?) so I am fine with it
And at least for this shop I sadly have experience with them (hello from low emission hell, on next day of "better stay indoor and do not open windows" warnings)
There was a problem hiding this comment.
low match score is as far as I know to keep gas station higher
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
Description, Motivation & Context
Terms were suggested by LLM and human-reviewed (>90% of suggestions were discarded for various reasons).
the main LLM command was:
add missing useful search terms very likely to be used by mappers to identify given POI when searching for presets in editors, using US English, if existing terms cover it do not add it. Especially if terms has multiple words and first of them is term already then value is really low. Search terms are added in 'terms' property.Related issues
none
Links and data
Relevant OSM Wiki links:
Relevant tag usage stats:
none
Checklist and Test-Documentation Template
Read on to get your PR merged faster…
Follow these steps to test your PR yourself and make it a lot easier and faster for maintainers to check and approve it.
This is how it works:
After you submit your PR, the system will create a preview and comment on your PR:
Once the preview is ready, use it to test your changes.
Now copy the snippet below into a new comment and fill out the blanks.
Now your PR is ready to be reviewed.