Hello,

This does not look to be the actual name of this area. Additionally, you removed the wetlands tag, which should not be done even if there was a name to add.
---
#REVIEWED_BAD #OSMCHA
Published using OSMCha: https://osmcha.org/changesets/162979557

Also, you don't need to split up your edits so much; you can do one changeset per group of houses instead of one changeset for every house.
---

Published using OSMCha: https://osmcha.org/changesets/162957473

Hello,

When editing, please try not to delete the original object only to remap it from scratch. Since you are using JOSM, you can install the plugin named "utilsplugin2" (see Edit>Preferences>Plugins), and then use the "Replace Geometry" tool. With that, you can move the original house out of the way, draw the new shape, select both the old and new shape, and then press Ctrl+Shift+G. That will keep the history of the old house while also allowing you to make a new shape.
---

Published using OSMCha: https://osmcha.org/changesets/162957473

Hi,

What is the source for the destination:lanes tags you added? I don't recall there being signs for the RFK bridge here.
---

Published using OSMCha: https://osmcha.org/changesets/161475541

Under the new trunk classification guidelines, the road could only be sufficiently important to justify trunk or motorway between the two other motorways.

Hello,

I see that you used the tag `lanes:unmarked=yes` here to denote the lack of lane markings. Currently, there are only 515 of this tag in the world, in contrast to `lane_markings=no` which has over 1.4 million. Also, the latter tag is documented on the wiki, while the former is not. As such, I would recommend using `lane_markings=*` for this going forward.

The two most common values for `lanes:unmarked=*` is 1 and 2, so perhaps this tag is usually used when some lanes are not marked while the rest are. Even still, this is a much more obscure tag than `lane_markings=*`.

I removed ref=GSPC because this is a simple abbreviation of the full name, rather than a code that was consciously assigned by the authorities.

Whoops, the tags were actually usually hgvtrimble=no.

Previously, I had used network=US:NY:Parkway:PIPC on a couple of these roads, as part of the 2022 NY Parkway cleanup. This was kind of a questionable value from the beginning, since the PIPC is an interstate compact. To reflect this, I have moved it up to the US: namespace, rather than having it below US:NY.

In addition, most of these initialisms are seldom used by the general public to refer to the roads. Even if they were used, they should go into another tag, such as short_name. I suspect that they were originally added into ref so that they would appear on Carto, which is tagging for the renderer.

Hi,

I believe that highway=motorway_junction nodes generally aren't supposed to have name=* values in the US; to express where the exit leads to, use destination=*, destination:street=*, destination:ref=*, etc.

I reverted this because I didn't hear back. Please feel free to comment if you disagree

Hello,

I appreciate your contribution here, but there are a few issues that I'd like to point out, so that you can avoid making them in the future.

First, please break up your changesets by geographic area. In this changeset, you made unrelated changes across the whole state of New York; these ideally should have been one changeset per city, approximately. This helps other mappers verify your work. Other mappers have informed you of this issue, but it seems like you have still been making large edits since.

Secondly, you changed a few road classifications in this changeset. Aside from this not being represented in the changeset comment, the roads that you upgraded to motorway should not be motorways. Please refrain from upgrading short segments of road to motorway, and try engaging with the community (see OSMUS Slack and Discord) before making such changes. Many states have their own guidelines which should be followed now.

The previous two issues were the most important ones here, but there are a couple more. You specified `damage:event=tornado` on way 442540277, without citing any sources. Please clarify what happened here, and how you know about it. The last thing is that descriptive names such as "Drive-in Cinema (disused)" should be avoided; "disused" should never go in `name`, and instead should be conveyed in either the opening_hours tag or by prefixing the amenity tag with `disused:`. If "Drive-in Cinema" is not the official name of the location, this should not go in the name either.

I appreciate you reading through these issues, and look forward to your response.

Thanks,
John

Hi,

This name is not signed anywhere along the road. Supposedly, it was once the commonly used name, but as a local I only ever hear it referred to as "route 17". As such, I believe that noname=yes with ref="NY 17" is the appropriate tagging for the road.

Note that this does not apply to NY-17 west of Binghamton, which is a separately constructed road that does have a signed name.

Hello,

This change is incorrect. Since there are traffic lights, traffic in the circle does not always have the right of way, and thus junction=roundabout is not appropriate. junction=circular is indeed the correct tag here, which is how it was tagged before.

Best,
John

It looks like you did, so I went ahead and fixed this one.

Hi,

It looks like you also made a mistake on this way: osm.org/way/956455770

Did you mean 45 mph, instead of 445 mph?

Hi there,

It looks like you made a mistake with the speed limit on this way: osm.org/way/826790148

Did you mean 35 mph instead of 435 mph?

Hi there,

It looks like you made a mistake with the speed limit on this way: osm.org/way/9276911

Did you mean 30 mph, instead of 302 mph?

I didn't think the Bear Mountain section was as important, so I left it as tertiary. If you disagree with any part of this changeset, please leave a comment here.