piebro/osm-data huggingface dataset:
EinKonstanzer (2021-08-19 17:48:54+00:00): Von derartigen Änderungen wird inzwischen dringend abgeraten. Der Hintergrund dafür ist die Update-Zyklus vieler Kartenanbieter bzw. der Update-Zyklus wie er halt in der Praxis aussieht.
Die Änderung hast du am 18.05.21 vorgenommen. Jemand der bis heute gewollt/ungewollt eine Karte vor dem Änderungsdatum benutzt hat natürlich nicht den aktuellen Stand.
Aber: (Die Sperrung geht bis 30.09) Leute die einen Kartenstand vom September noch im Dezember benutzen (gewollt/ungewollt) haben eine Karte in der es definitiv einen Fehler gibt. Einen Fehler der fatale Auswirkungen auf das Routing haben kann. Eine temporäre Sperrung ist dagegen ein Schönheitsfehler und eine Herausforderung für den Fahrer... - aber auch nicht wirklich, da es lokal eine Umleitung gibt.

Die Lösung für sowas ist das "conditional" tagging für den "access".
Siehe:  https://wiki.openstreetmap.org/wiki/DE:Key:access#Zeitabh.C3.A4ngige_und_bedingte_Beschr.C3.A4nkungen

PS: Ein weiteres Problem bei der aktuellen Praxis ist, daß das Rückgäng machen der Änderung zum Ende der Sperrung dann aber häufig vergessen wird... Dann haben wir nämlich wirklich einen ernsten Kartenfehler.
EinKonstanzer (2021-08-19 17:50:23+00:00): Ein Beispiel das ich gerade gefunden habe...
access:conditional = no @ (2021 Aug 13-2021 Oct 13)
Statt "access" auch "vehicle", "motor_vehicle", ... je nach Grad der Sperrung
lubaum (2021-08-20 07:59:11+00:00): Vielen Dank für Deine Hinweise. Mir ist sind die damit verbundenen Problematiken durchaus bewußt und ich bin auch selbst einmal über eine "vergessene" Sperrung der Autobahn bei Leipzig gestolpert... Und wenn wir von Kartenanbietern nur blind benutzt werde ohne meine "description" anzuschauen - selber schuld.
Das Taggen mittels access:conditional ist mir bekannt, aber: Leider ist (oder: war bisher?) unser OSM-System nicht fähig, das geeignet auszuwerten: der Renderer zeigt auch nach Ablauf des Sperrzeitraums weiterhin eine gesperrte Straße  und der Router OSRM sagte mir in Gestalt von Michael S.(datendelphin) er fände es erfreulich, wenn ich das im OSRM programmieren würde........
Sinnvoll hielte ich einen Austausch im Forum zu dieser Thematik. Zu meiner Arbeit hier im Landkreis Kn-Si: Ich bekomme vom Landratsamt solche Meldungen und versuche dann, die zugehörigen Kommunen, die ja osm zu nutzen beginnen, in die Verantwortung der Aufhebung solcher Sperren einzubinden.
LG Ludwig B
lubaum (2021-08-20 09:03:02+00:00): wir waren ja schon einmal in Kontakt, wegen eines Fußwegs in Steißlingen, meiner Heimatgemeinde. Auch dieses Mal bin ich Dir sehr dankbar dafür. Ich kann da sicher noch vieles besser machen und von Dir lernen.
Da Du offensichtlich an Deiner osm-Heimat  immer noch aktives Interesse hast, möchte ich den weiteren Austausch in unsere Talk-Liste talk-bodensee@openstreetmap.org fortsetzen und Dich einladen, diese zu abonnieren (ist dafür besser geeignet als das Forum) talk-bodensee@openstreetmap.org
LG ludwig
EinKonstanzer (2021-08-20 18:49:25+00:00): Das "conditional" bedeutet im Prinzip erst einmal kein klassisches "access" zu verwenden. So weit ich informiert bin wertet der OSM-Carto-Stil (osm.org) "conditional" nicht aus - bleibt also alles wie es ist.
"conditional" wird z.B. auch für Straßen benutzt, die z.B. von 09:00 - 16:00 Einbahnstraße sind.
Dynamisches Rendern ist also nicht nur in weiter ferne sondern auch mit einem großen Fragezeichen behaftet.

So weit ich informiert bin unterstützt im Moment sogut wie gar kein Router das "conditional". OSMand als Routing-App tut es wohl... Das spricht natürlich für die klassische Lösung.
osmo777 (2021-08-23 13:59:45+00:00): Прорисованные по трекам тропинки упрощены и спрямлены. Болота удалены. Безосновательно заменены residental на service. Вредительство.
kakiremora (2021-05-18 21:28:33+00:00): użyłem tu też rzeźby terenu ISOK
gabro00 (2021-05-19 07:17:29+00:00): Szia!
A kút felvételénél már jeleztem Tamásnak, hogy ez nem a Füvészkert :)
https://www.openstreetmap.org/changeset/89894945
Sikerült azóta megtudni mit válaszolt a bejelentő?
user_5359 (2021-05-19 06:42:23+00:00): Welcome to OSM! Short inquiry to the key service_times: the services are held every day or only on Sundays? In the latter case add a Su (followed by space character) at the beginning of the value. Please notice also: We use the 24h notation for time informations.
By the way: The 2nd floor isn't noramly a addr:unit we use the tag level=2
woodpeck_repair (2021-05-18 22:15:57+00:00): This changeset reverts some or all edits made in changesets 104907832, 104907986, 104907996, 104908019, 104908040, 104908063, 104908083, 104908141, 104908158, 104908170, 104908180, 104908206, 104908227, 104908243, 104908262, 104908274, 104908283, 104908333, 104908339, 104908372, 104908385, 104908404, 104908422, 104908437, 104908474, 104908480, 104908505, 104908527, 104908549, 104908567, 104908592, 104908614, 104908627, 104908636, 104908657, 104908669, 104908683, 104908696, 104908709, 104908722, 104908733, 104908739, 104908748, 104908780, 104908791, 104908801, 104908814, 104908824, 104908841, 104908863, 104908874, 104908896, 104908904, 104908924, 104908930, 104908944, 104908959, 104908969, 104908986, 104908997, 104909018, 104909032, 104909070, 104909083, 104909100, 104909109, 104909127, 104909172, 104909183, 104909208, 104909235, 104909251, 104909260, 104909289, 104909313, 104909331, 104909344, 104909362, 104909370, 104909390, 104909409, 104909430, 104909450, 104909460, 104909472, 104909486, 104909500, 104909515, 104909546, 104909563, 104909584, 104909613.
Enock4seth (2021-05-19 07:53:07+00:00): What are you trying to achieve by naming buildings "Q"? Please stop putting funnys text in the name field for features. This is very bad and kindly avoid it. Read https://wiki.openstreetmap.org/wiki/Key:name
Jakka (2021-05-19 06:27:57+00:00): Indien mogelijk een foto aan toevoeging via een eenvoudige app https://mapcomplete.osm.be/bookcases.html?#welcome thx
            ---
            #REVIEWED_GOOD #OSMCHA
            Published using OSMCha: https://osmcha.org/changesets/104917137
user_5359 (2021-05-21 06:23:59+00:00): Hello! Please have look on http://www.osm.org/way/203895616

Please read before https://wiki.openstreetmap.org/wiki/Key:height#Height_of_buildings

What is the translation of the roof:material value?
0tofoto (2021-05-22 00:32:38+00:00): I read this - https://wiki.openstreetmap.org/wiki/Simple_3D_Buildings

Seems eternit roof would be more accurate but such brand is not known locally - https://wiki.openstreetmap.org/wiki/Key:roof:material
user_5359 (2021-05-22 02:14:24+00:00): You have a typo (buiding:height) in you keys. But building:height should be replace with height.

By the way, is OSM a local or a worldwide project?
0tofoto (2021-05-23 15:10:51+00:00): Thanks for notice but can't find the typo there is no such tag - https://taginfo.openstreetmap.org/keys/buiding

Height is overall height but building:height height of building only. 

OSM is worldwide project so would like to avoid local brand names. Not all dolls are Barbies. 
More correct name of roof material would be Asbestos cement sheet or fibrolite roof.
user_5359 (2021-05-23 17:08:25+00:00): Please look on https://www.openstreetmap.org/way/203895616/history. You added version 2, user mueschel correct his typo.

Please read also https://wiki.openstreetmap.org/wiki/Key:height#Height_of_buildings. You don‘t need this information.
ovruni (2021-05-19 04:36:49+00:00): Hola, en este conjunto de cambios se puso al Asilo de ancianos como edificio, sin embargo la etiqueta de edifcios se encuentra destinada para los edificios individuales, los cuales se encuentran dentro de esta área. Saludos
mariotomo (2021-05-19 22:38:41+00:00): hola.  JOSM no te advierte que estás subiendo líneas sin etiquetar? 944469110, 944469101 (es algo que haces con alguna frecuencia).
Warin61 (2022-10-12 00:17:39+00:00): Redfern Train Staion:

Train lines are at layer 0. 

Platforms should be at the same layer.

The walk ways go over the train line .. so they should be at layer=1 - same as the road way and cycle way ... 

I have attempted to change this to where it should be.
ElliottPlack (2021-06-23 13:27:15+00:00): Hi M86! I work for MDOT State Highway Administration (eplack@mdot.maryland.gov) and we're responsible for the inventory and  maintenance of park & rides in Maryland. By adding this to OSM, you've done us a great service because it already appears on some maps we use. Thank you!! We just sent a crew out today to take some measurements. I'll update OSM with as much as I can from that collection. Let me know if you have any questions, comments, or concerns. -Elliott
dieterdreist (2021-12-06 12:19:08+00:00): Hi, I just noticed you have created a relation for the "Eurozone". Maybe I am misgiuded, but from my understanding, this could be automatically computed by evaluating the low profile "currency" tags and combining them with the official boundaries of the European Union? The result would be all countries of the EU which have the Euro as their official currency. Isn't this good enough? 
Geneally, we are striving for the KISS principle, the simpler a representation, the better. Relations are heavy weight. Their use should be reduced to the minimum (cases where there are unavoidable to express what is wanted).
I propose to remove this relation.
Cheers,
Martin
JeroenvanderGun (2021-12-06 20:15:25+00:00): This relation has only 19 members so it's not heavy-weight. (This is not a type=boundary relation.) It's also useful to have this relation because it attaches a name and metadata to the query, e.g. making it retrievable by search.

That said, you indirectly do raise a valid question regarding the proper definition of the underlying query: I'm unable to find an authoritative source regarding whether Overseas Countries and Territories are to be considered part of the Eurozone territory. If unable to find an exact definition of the Eurozone, it's probably better to remove this relation.
dieterdreist (2021-12-06 21:51:31+00:00): Hi Jeroen, thank you for your reply. The “weight” of the relation is not its immediate complexity (I agree, 19 members are easy to overlook, and it won’t produce many versions because they are all relations). But if you look at a typical operation with OpenStreetMap data like geographic filtering to get a regional extract, if you are aiming at geometric completeness you will have to traverse all the relations, then all their ways to finally get all the nodes. Everybody who creates an extract with some country border in it and who wants to avoid corrupted relation geometries will have to do this parsing (will do it automatically), these are thousands of “2nd level members” (members of the relations that are members of this relation). In principle I can understand the desire for “hardcoded named queries”, but it doesn’t scale if we do it with relations in the main database, because all the mappers will have to deal with these relations as separate geometric objects (there are also NATO and similar relations) when in reality a list of countries is probably sufficient
in this case.
Cheers Martin
JeroenvanderGun (2021-12-06 22:24:16+00:00): As far as I understand, nesting relations inside larger relations is common mapping practice: https://wiki.openstreetmap.org/wiki/Super-relation

For example, this relation indirectly contains the entire main road network of Europe: https://www.openstreetmap.org/relation/7884303

Surely any data processing tool will already have appropriate limitations in place for dealing with nested relations. The Eurozone relation was created 7 months ago and there have been zero complaints about crashing computers. Both the E-road network relation and the main European Union relation exist much longer than that. There are probably many more examples.
dieterdreist (2021-12-06 22:50:18+00:00): indeed, those network relations like https://www.openstreetmap.org/relation/7884303 or those “collecting” public transport routes are also questionable.
https://wiki.openstreetmap.org/wiki/Relations_are_not_categories
dieterdreist (2021-12-06 22:51:32+00:00): it’s not about crashing computers, it’s about additional processing for redundant data.
Luzandro (2021-05-21 17:47:30+00:00): Hi,
please use separate changesets for different countries. It's more comprehensible what you did and you don't show up in the feeds of people on multiple continents, who are looking for changes in their region.
thx
            ---
            
            Published using OSMCha: https://osmcha.org/changesets/104919815
mariotomo (2021-05-19 22:00:18+00:00): hola, gracias por mejorar el mapa de la comunidad OSM y si podrías ponerle un poco de atención al comentario, que explique más, sin deber uno entrar en https://resultmaps.neis-one.org/osm-change-viz?c=104919866
ravali_78 (2021-05-20 05:32:58+00:00): Hi vydn,
I have fixed the alignment,as building polygon overlapping with service highway.Kindly check with way 944525086,https://www.openstreetmap.org/changeset/104997251.
Thank you.
            ---
            
            Published using OSMCha: https://osmcha.org/changesets/104919921
user_7130456 (2021-06-29 15:22:59+00:00): 新人你好，下次绘图的时候麻烦稍微看清楚你选择的是工业用地还是工业建筑物，建筑不是用地
mariotomo (2021-05-19 21:54:12+00:00): hola.  en qué sentido, "nomenclatura"?
mariotomo (2021-05-19 21:53:06+00:00): hola. veo que estás añadiendo nombre "numéricos" en forma textual. es un asunto que he propuesto en el foro#61: https://forum.openstreetmap.org/viewtopic.php?id=72599
user_5359 (2021-05-19 06:38:51+00:00): Welcome to OSM! Please notice this is a production database not a playground or testing database. I revert your entry. If this is a misunderstanding, you can answer with your arguments here.
user_5359 (2021-05-21 05:10:03+00:00): Hello! What is the meaning of "building industrial"="source cidi"? Did you mean "building"="industrial" and"source"="cidi"? But I can't see any building on the aerial images. And don't add building to a property.

See http://overpass-turbo.eu/s/17zU
mariotomo (2021-05-19 21:49:15+00:00): https://www.openstreetmap.org/way/944489991 lo dejaste sin etiquetas.
user_5359 (2021-05-19 06:36:36+00:00): Welcome to OSM! Don't add any private information from residential or owner to a residential building!
JessAk71 (2021-05-19 18:09:20+00:00): I went ahead and corrected the name on the poi for Riley_Andersen, it looks like it also needed an amenity tag. Funny enough via a google search it looks like Riley_Andersen tagged this poi for the house of the main character in Inside Out... regardless it still should not be tagged as such...
ravali_78 (2021-05-20 03:04:53+00:00): Hi emer_enriquez,
I have fixed the alignment,as building polygon overlapping with service highway.Kindly check with way 500720548,https://www.openstreetmap.org/changeset/104990735.
Thank you
            ---
            #REVIEWED_BAD #OSMCHA
            Published using OSMCha: https://osmcha.org/changesets/104922548
user_5359 (2021-05-19 06:35:47+00:00): Welcome to OSM! Please have a look on https://www.osm.org/way/944493553: This should be a garage with a capacity of two cars? I think this is a normal residential building. 
Don't add any private information from residential or owner to a residential building!
By the way, we only indicate the capacity if it is a publicly accessible garage.
mariotomo (2021-05-19 21:25:30+00:00): para las próximas que te encuentres a mapear: las iglesias cuadrangulares caben en la denomination= foursquare / pentecostal.
ravali_78 (2021-05-20 02:55:37+00:00): Hi Villnorv,
I have fixed the alignment,as building polygon overlapping with residential highway.Kindly check with way 886881012,https://www.openstreetmap.org/changeset/104990507.
Thank you.
            ---
            
            Published using OSMCha: https://osmcha.org/changesets/104922803
topius (2021-05-19 09:15:19+00:00): Villages should be mapped with a node at the center, and then map the area taken up by the village as "landuse=residential". And smaller sub-villages like Baiman, Babaten, Uriyasi, etc. consisting only of a few houses should be tagged as hamlets, not villages. In addition, the name of the village should be just the name, without "village" (so just Kalalen, not Kalalen village).
Kovoschiz (2021-05-19 07:13:59+00:00): Please be extra cautious and don't remove things you haven't comprehended fully. `place=locality` has 2 use case here. 
1. This may be for the demolished settlements around the Reservoir moved to the foothills. 
2. Historically they would be yet-to-be established `natural=` tags viz `=ridge`, `=mountain_range`. 
So this at least needs to be checked whether it should moved to a correct `=peak` location.
Herman Lee (2021-07-11 15:32:26+00:00): 这位大哥，您可以不用这样种树的[捂脸]，广州的树挺多的，这样似乎意义不大。。。
Mok180 (2021-07-13 04:20:37+00:00): 其实我主要更新建筑，树是偶尔顺手的[捂脸]
eggie (2021-05-19 04:47:27+00:00): reverted by changeset #104925678
rthik007 (2021-05-20 06:24:32+00:00): Hi Emy Capila,
Fixed the alignment, as building polygons overlapping with residential road WAY: 51671074
Kindly check with
https://www.openstreetmap.org/changeset/105001601
Thank you.
            ---
            #REVIEWED_BAD #OSMCHA
            Published using OSMCha: https://osmcha.org/changesets/104923868
ovruni (2021-05-19 04:48:52+00:00): In the INEI database this street is Santa Veronica. What are your sources for renaming this street? If you have any questions about the mapping in Peru, you can do so through the telegram channel https://t.me/osmpe or the mailing list of the OSM community in Peru https://lists.openstreetmap.org/listinfo/talk-pe
user_2066394 (2021-05-19 10:05:23+00:00): Da wäre es sinnvoller gewesen, den ursprünglichen Knoten einfach zu verschieben.
Attribute wie
tourism=museum
website=*
wheelchair=*
sollten schon mitübertragen werden.
bhamidpour (2023-07-30 05:38:51+00:00): the peak name is Kamki for ever! not kiyamaki!! you can see its name on peak sign clearly !!!
Reza Babri (2023-07-30 05:56:28+00:00): سلام آقای حمیدپور
ممنون از توضیحی که دادید
من نگاه کردم هر دو اسم کیامکی و کمکی در این نقشه ثبت شده. منتها چون او اس ام قوانین خودش رو داره باید فقط یکی از اسامی در تگ
name
ثبت بشه. اسم کمکی در تگ
alt_name
ثبت شده.
abakus_uploader (2021-05-19 22:31:09+00:00): (Wiadomość automatyczna)
Cześć!
Otrzymujesz ten komentarz, ponieważ dodałeś lub zmodyfikowałeś adres, który jest niepełny lub błędny.
Adres powinien zawierać minimum trzy znaczniki: addr:city=[nazwa_miejscowosci] + addr:street=[nazwa_ulicy] + addr:housenumber=[numer_porządkowy].
Wyjątkiem są miejscowości bez nazwanych ulic - wtedy stosujemy znaczniki: addr:place=[nazwa_miejscowosci] + addr:housenumber=[numer_porządkowy].
Bez poprawnego oznaczenia adresów, nie będą one brane pod uwagę przez wyszukiwarki, dlatego warto je dostosować do ww. schematu (w zależności od sytuacji).
Więcej informacji na forum: https://forum.openstreetmap.org/viewtopic.php?id=5632 - a pytania najlepiej zadawać tu: https://forum.openstreetmap.org/viewtopic.php?pid=681873
Pozdrawiamy!
waterced (2025-09-22 17:55:25+00:00): Bonjour, à l'occasion de ce changetset, vous avez ajouté ce rond-point en construction https://www.openstreetmap.org/way/944513458 Or, 4 ans plus tard, aucune trace de travaux. Certes le cadastre montre que les parcelles sont prévues, ne faudrait-il passer passer ces rues en "proposed"
ovruni (2021-05-19 06:49:35+00:00): This street was intentionally named "Calle Los Gladioles" as part of the campaign to add streets that were eliminated due to the reversion in 2018. Since you have not responded to previous messages, I have re-named and mentioned about your editions in the telegram channel in the community in Peru. Please communicate with the community in Peru through the telegram channel https://t.me/osmpe or the mailing list https://lists.openstreetmap.org/listinfo/talk-pe
gscscnd (2021-05-19 06:53:03+00:00): IMHO na <https://www.openstreetmap.org/way/944521341> powinno być service=spur.
Rizki IRM-ED (2022-12-02 02:29:04+00:00): Hello panja007. Thanks for your contribution. I delete the road (Way 944529138) because not connected to the nearest node and there is already a road next to it that is connected to a residential highway (https://maproulette.org/challenge/30179/task/142913985). Thank you.
Jennifer Jacobsen-Wood (2021-05-19 21:46:37+00:00): Thank you so much! I really appreciate the assistance.

There are a few other Peoria area streets that have have been vacated and still have nodes despite now being private property. I’ll have to take the time to make notes on the map to have them resolved.
abakus_uploader (2021-05-19 22:31:10+00:00): (Wiadomość automatyczna)
Cześć!
Otrzymujesz ten komentarz, ponieważ dodałeś lub zmodyfikowałeś adres, który jest niepełny lub błędny.
Adres powinien zawierać minimum trzy znaczniki: addr:city=[nazwa_miejscowosci] + addr:street=[nazwa_ulicy] + addr:housenumber=[numer_porządkowy].
Wyjątkiem są miejscowości bez nazwanych ulic - wtedy stosujemy znaczniki: addr:place=[nazwa_miejscowosci] + addr:housenumber=[numer_porządkowy].
Bez poprawnego oznaczenia adresów, nie będą one brane pod uwagę przez wyszukiwarki, dlatego warto je dostosować do ww. schematu (w zależności od sytuacji).
Więcej informacji na forum: https://forum.openstreetmap.org/viewtopic.php?id=5632 - a pytania najlepiej zadawać tu: https://forum.openstreetmap.org/viewtopic.php?pid=681873
Pozdrawiamy!
eggie (2021-05-19 06:58:38+00:00): Welkom bij OpenStreetMap. Ik heb een uitgebreid welkomstbericht met wat handige tips en links gestuurd als privebericht. 

Feedback.. Alle gebouwen zijn directe import vanuit het Kadaster. Beter geen gebouwen op gebuwen intekeken. Dat geeft een foutmelding.
Het beste kun je de adresnodes gebruiken om info bij te taggen.
De tag voor een faculteit is amenity=university

MVG, 


Je edits https://overpass-api.de/achavi/?changeset=104932293
Eggie 

Namens de OSM Community
eggie (2021-05-19 07:05:35+00:00): Gebouw verwijderd.. info overgezet op bestaande gebouw en adresnode nr 40.
Merged info with present addressnode and present building. 
mvg
egggie
Warin61 (2021-05-20 07:04:42+00:00): Hi,
Welcome to OSM! There was a small mistake thagt damaged the platform Way: Stop 52: River Street (165183150).  This has been repaired by a partial reversion of this changeset to restore the platform.
Alliegaytor (2025-04-11 04:06:27+00:00): Hi alisaluk, just letting you know that platform n4427569779 was also mistakenly moved in this changeset. I've restored the node's location in #164795619

Happy mapping!
silence2001 (2021-07-14 09:28:08+00:00): Moin Andy,

Du hast den die Abbiegebeziehung vom Hohenzollernring nach links in die Bernadottestraße entfernt. An diesem Punkt biegen sämtliche Buslinien, die regulär über die Bleickenallee fahren seit den Baumaßnahmen im Hohenzollernring legal nach links ab, um ihren Umleitungsweg über die Holländische Reihe zu fahren. Wie passt das zusammen? Zumindest müsste eine Ausnahmeregel für den ÖPNV eingepflegt werden, da diese Baumaßnahme noch mindestens bis Dezember 2021 andauert.

Viele Grüße

Jan
mavl (2021-05-20 16:18:58+00:00): Здравствуйте. Этот пакет правок удаляет правильные данные. Эта проблема уже обсуждалась в [https://www.openstreetmap.org/changeset/104566429]. В этом пакете правок удалён дом с адресом:  
  
https://www.openstreetmap.org/way/655381232/history  
  
На его место добавлен дом, у которого адрес добавлен без указания улицы:  
  
https://www.openstreetmap.org/way/944536500  
  
В результате испорчен адрес.  
К сожалению, у меня нет времени сейчас проверять все ваши изменения. Напишите, пожалуйста, как вам будет удобнее: вы проверите свои изменения самостоятельно и восстановите испорченные адреса или мне полностью откатить этот пакет правок?  
  
Просто для информации: если у здания неточные расположение или конфигурация, то это здание нужно переместить в правильное место или изменить его конфигурацию на правильную. Удалять здание не нужно. Добавлять на его место новое здание не нужно.
user_5359 (2021-05-19 12:14:54+00:00): Hello! Please use the key name instead of Name (I have already corrected this)