Kinsio's Comments
| Changeset | When | Comment |
|---|---|---|
| 165310355 | 8 months ago | I just noticed some of your zip_left and zip_right tags differed from the tiger:zip_left and tiger:zip_right tags. What was your data source for this information? |
| 165349269 | 8 months ago | turn:lanes:both_ways=1 is incorrect tagging. What you're looking for is lanes:both_ways=1 + turn:lanes:both_ways=left. (I've corrected the instances of the error in this changeset, but just for future reference.)
|
| 165348123 | 8 months ago | Agh, I'm so embarrassed 😆 Nice catch, thanks!
|
| 130635620 | 8 months ago | I just fixed several ways that were tagged access=private in this changeset. Private access is only appropriate for cases where access requires explicit permission on an individual basis. For commercial areas where access should be expected for people visiting to patronize the business (in these cases, guests staying at the hotels), the correct tagging is access=customers. |
| 165059298 | 8 months ago | Thank you for addressing my fixme, but for future reference, it seems shop=printing is the preferred tag for this kind of shop. Fixed here in changeset #165091411.
|
| 158541893 | about 1 year ago | Did you read my changeset description wrong? I don't think we're in disagreement here. They were hamlets before, I changed them to neighbourhoods. |
| 154548532 | over 1 year ago | Possibly debatable but it is highlighted by Osmose as an issue: <https://github.com/osm-fr/osmose-backend/blob/dev/plugins/TagFix_MultipleTag2.py>. The rationale is one feature, one OSM element: as the generated Osmose issue says, the amenity is under the roof, not part of it. Maybe didn't need to be Fixed now that I know about how that's kinda part of the broader amenity on building debate, but I did these to resolve the Osmose issues at the time. |
| 155076376 | over 1 year ago | That's what I did, yeah. As for users who have it enabled, it's disabled by default and the warning for enabling it mentions that it's "not useful in some regions" so I think that's basically as covered as we can reasonably expect. |
| 155076376 | over 1 year ago | For future reference, I think around here if it's air_conditioning=yes then it's probably better to just leave it unspecified. It's not really useful information because it's basically expected for any given place.
|
| 155489595 | over 1 year ago | This information would be better provided in the source tag. The changeset comment should describe what you changed. For more info on writing good changeset comments see osm.wiki/Good_changeset_comments
|
| 155157745 | over 1 year ago | Heck, I wrote most of that comment earlier today and didn't think to refresh the page before submitting it just now 😠Didn't mean to just reiterate something already said, hopefully my link is at least helpful |
| 155157745 | over 1 year ago | It would also be helpful in the future to separate your changes in different distant areas into different changesets. For this one, the California, USA edits could have been one changeset and the Paraná, Brazil edits could have been another. See osm.wiki/Changeset#Geographical_size_of_changesets |
| 113058966 | over 1 year ago | Allison: I've explained a bit more in my comments on changeset #113064020. I selected this changeset and the others I'm reverting because I compared the building outlines to the original dataset from Microsoft and the vast majority, if not all, of them are an exact match. Selecting the entire layer and pressing "add" is in fact what seems to have been done here. |
| 113064020 | over 1 year ago | > Yes, MapWithAI does throw an occasional "bum steer" out there but the vast majority of times it's good data that only needs relatively minor adjustments.
And yeah, I should've tried to contact you earlier, you're right, but this is still a violation of the import guidelines, which I'm perfectly within my rights to revert. I encourage you to compare the more recent addition of buildings by nitemoon: while they also used MapWithAI to add a lot of buildings, every single building has at least some kind of adjustment so it's clear that they inspected each one individually. That's allowed without discussion because it's ultimately no different than having drawn the building yourself, but just hitting "add" over and over without checking the buildings is not. That's a data import, which requires an import plan and discussion. |
| 113064020 | over 1 year ago | And to explain a bit more on why I decided to revert, it's because I keep running into these buildings when I'm editing in the area and I personally hate adjusting these things. I'd rather just draw them by hand to begin with and not have to worry about rotating. That was my motivation to start investigating how they got here. |
| 113064020 | over 1 year ago | Hi, sorry, I should have been a bit more diplomatic about how I started reaching out here and explained a bit more about what I'm doing and my rationale. To answer the second question, because I just recently got involved with OSM :) As for them not being an import because you used MapWithAI, not necessarily. I've verified that the buildings added in the changesets I'm examining for reverting were entirely or almost entirely directly copied from, not based on and then adjusted from, the Microsoft building footprint data release of June 2018. This constitutes an import and is therefore subject to the usual import guidelines. I'm leaving the fixme tags (which I did notice you've started working on, thank you) on buildings that aren't suitable for outright deletion because they have had meaningful information added to them, and anything that comes up when I'm going through the changesets that I can see there was some attempt to adjust the geometry on I'm leaving alone. I appreciate the intent to improve the building coverage around here but this is not the way to do it. If you want to improve building coverage in the area by directly adding the Microsoft building footprints as a starting point, it needs to be carried out as a proper import. And for what it's worth, I live in Beaumont, thank you for the work you've done mapping around here, I do appreciate it! |
| 113064020 | over 1 year ago | I reverted this undiscussed mass import with my changeset/154791445. |
| 113058966 | over 1 year ago | Sorry, had the wrong thing selected when I was checking the number. changeset/154780652 actually. |
| 113058966 | over 1 year ago | I reverted this undiscussed mass import with my changeset/154794340. |