osm_poi_matchmaker.libs.file_output.generate_osm_xml()

Complexity

Conditions

90

Size

Total Lines	265
Code Lines	215

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

# -*- coding: utf-8 -*-
def generate_osm_xml(df, session=None):

Argument name "df" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

This function exceeds the maximum number of variables (38/15).

    """Crete OpenStreetMap (OSM XML) file from passed Panda Dataframe

    Args:
        df ([type]): [description]
        session ([type], optional): [description]. Defaults to None.

    Returns:
        [type]: [description]
    """
    db = POIBase('{}://{}:{}@{}:{}/{}'.format(config.get_database_type(), config.get_database_writer_username(),

This line is too long as per the coding-style (112/100).

Variable name "db" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                                              config.get_database_writer_password(),
                                              config.get_database_writer_host(),
                                              config.get_database_writer_port(),
                                              config.get_database_poi_database()))
    pgsql_pool = db.pool
    session_factory = sessionmaker(pgsql_pool)
    Session = scoped_session(session_factory)

Variable name "Session" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

    session = Session()
    osm_xml_data = etree.Element('osm', version='0.6', generator='JOSM')

Module 'lxml.etree' has no 'Element' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

    default_osm_id = -1
    current_osm_id = default_osm_id
    added_nodes = []
    try:

Too many nested blocks (10/5)

Too many nested blocks (6/5)

Too many nested blocks (7/5)

        for index, row in df.iterrows():

Possibly unused variable 'index'

            tags = {}
            osm_live_tags = {}
            main_data = {}
            current_osm_id = default_osm_id if row.get('osm_id') is None else row.get('osm_id')
            osm_version = '99999' if row.get('osm_version') is None else row.get('osm_version')

Possibly unused variable 'osm_version'

            if row.get('osm_node') is None or row.get('osm_node') == OSM_object_type.node:
                try:
                    josm_object = 'n{}'.format(current_osm_id)
                    main_data = etree.SubElement(osm_xml_data, 'node', add_osm_node(current_osm_id, row))

This line is too long as per the coding-style (105/100).

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    logging.exception('Exception occurred')
            elif row.get('osm_node') is not None and row.get('osm_node') == OSM_object_type.way:
                try:
                    main_data = etree.SubElement(osm_xml_data, 'way', add_osm_way(current_osm_id, row))

This line is too long as per the coding-style (103/100).

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                    josm_object = 'w{}'.format(current_osm_id)
                # Add way nodes without any modification)
                    node_data = []
                    for n in row.get('osm_nodes'):

Variable name "n" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                        data = etree.SubElement(main_data, 'nd', ref=str(n))

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

Possibly unused variable 'data'

                    if session is not None:
                        # Go through the list except the last value (which is same as the first)
                        for n in row.get('osm_nodes'):

Variable name "n" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                            # Add nodes only when it is not already added.
                            if n not in added_nodes:
                                added_nodes.append(n)
                                way_node = db.query_from_cache(n, OSM_object_type.node)
                                if way_node is not None:
                                    node_data = etree.SubElement(osm_xml_data, 'node',

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                                                                 list_osm_node(n, way_node, 'osm'))
                                    if node_data.get('osm_live_tags') is not None and \
                                       node_data.get('osm_live_tags') != '':
                                        node_osm_live_tags = node_data.get('osm_live_tags')
                                        for k, v in sorted(node_osm_live_tags).items():

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                                            xml_node_tags = etree.SubElement(node_data, 'tag', k=k, v='{}'.format(v))

This line is too long as per the coding-style (117/100).

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                except TypeError as e:

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    logging.warning('Missing nodes on this way: %s.', row.get('osm_id'))
                    logging.exception('Exception occurred')
            elif row.get('osm_node') is not None and row.get('osm_node') == OSM_object_type.relation:

This line is too long as per the coding-style (101/100).

                try:
                    main_data = etree.SubElement(osm_xml_data, 'relation', add_osm_way(current_osm_id, row))

This line is too long as per the coding-style (108/100).

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                    josm_object = 'r{}'.format(current_osm_id)
                    relations = relationer(row.get('osm_nodes'))
                    for i in relations:
                        data = etree.SubElement(main_data, 'member', type=i.get('type'), ref=i.get('ref'),

This line is too long as per the coding-style (106/100).

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                                                role=i.get('role'))
                except TypeError as e:

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    logging.warning('Missing nodes on this relation: %s.', row['osm_id'])
                    logging.exception('Exception occurred')
            # Add already existing node, way, relation OpenStreetMap reference as comment
            try:
                if current_osm_id > 0:
                    osm_xml_data.append(etree.Comment(add_osm_link_comment(current_osm_id, row.get('osm_node'))))

This line is too long as per the coding-style (113/100).

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                # Add original POI coordinates as comment
                comment = etree.Comment(' Original coordinates: {} '.format(row.get('poi_geom')))

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                osm_xml_data.append(comment)
                if 'poi_distance' in row:
                    if row.get('poi_distance') is not None:
                        comment = etree.Comment(' OSM <-> POI distance: {} m'.format(row.get('poi_distance')))

This line is too long as per the coding-style (110/100).

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                    else:
                        comment = etree.Comment(' OSM <-> POI distance: Non exist')

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                    osm_xml_data.append(comment)
                if 'poi_good' in row and 'poi_bad' in row:
                    comment = etree.Comment(' Checker good: {}; bad {}'.format(row.get('poi_good'), row.get('poi_bad')))

This line is too long as per the coding-style (120/100).

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                    osm_xml_data.append(comment)
                # Using already definied OSM tags if exists
                if row.get('osm_live_tags') is not None:
                    tags.update(row.get('osm_live_tags').copy())
                    logging.critical(row.get('osm_live_tags'))
                    osm_live_tags.update(row.get('osm_live_tags').copy())
                # Adding POI common tags
                if row.get('poi_tags') is not None:
                    tags.update(row.get('poi_tags'))
                # Save live name tags if preserve name is enabled
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                if row.get('preserve_original_name') is True:
                    preserved_name = tags.get('name')
            except KeyError as e:

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.debug('No name tag is specified to save in original OpenStreetMap data.')
            try:
                # Overwriting with data from data providers
                for k, v in POI_TAGS.items():

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    if row.get(k) is not None:
                        tags[v] = row.get(k)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                if config.get_geo_alternative_opening_hours():
                    alternative_oh_tag = config.get_geo_alternative_opening_hours_tag()
                    # Alternative opening_hours handling for COVID-19 code path
                    if tags.get('opening_hours') is not None and tags.get('opening_hours') != '':
                        if row.get('poi_opening_hours') is not None and row.get('poi_opening_hours') != '':

This line is too long as per the coding-style (107/100).

                            if tags.get('opening_hours') == row.get('poi_opening_hours'):
                                tags[alternative_oh_tag] = 'same'
                            else:
                                tags[alternative_oh_tag] = row.get('poi_opening_hours')
                    else:
                        if row.get('poi_opening_hours') is not None and row.get('poi_opening_hours') != '':

This line is too long as per the coding-style (107/100).

                            tags['opening_hours'] = row.get('poi_opening_hours')
                            tags[alternative_oh_tag] = 'same'
                else:
                    # Alternative opening_hours handling for NON COVID-19 code path: just simply add opening_hours to tags

This line is too long as per the coding-style (122/100).

                    if row.get('poi_opening_hours') is not None and row.get('poi_opening_hours') != '':

This line is too long as per the coding-style (103/100).

                        tags['opening_hours'] = row.get('poi_opening_hours')
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # If we got POI phone tag use it as OSM contact:phone tag
                if row.get('poi_phone') is not None and row.get('poi_phone') != '':
                    tags['contact:phone'] = row.get('poi_phone')
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # If we got POI website tag use it as OSM contact:website tag
                if row.get('poi_url_base') is not None and row.get('poi_website') is not None:
                    if row['poi_url_base'] in row.get('poi_website'):
                        # The POI website contains the base URL use the POI website field only
                        tags['contact:website'] = clean_url('{}'.format((row.get('poi_website'))))
                    else:
                        # The POI website does not contain the base URL use the merged base URL and POI website field

This line is too long as per the coding-style (117/100).

                        tags['contact:website'] = clean_url('{}/{}'.format(row.get('poi_url_base'), row.get('poi_website')))

This line is too long as per the coding-style (124/100).

                # If only the base URL is available
                elif row.get('poi_url_base') is not None:
                    tags['contact:website'] = row.get('poi_url_base')
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # Short URL for source
                if row['poi_url_base'] is not None:
                    source_url = 'source:{}:date'.format(row.get('poi_url_base').split('/')[2])
                else:
                    source_url = 'source:website:date'
                tags[source_url] = '{:{dfmt}}'.format(datetime.datetime.now(), dfmt=DATE_FOTMAT)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # Write back the saved name tag
                if 'preserved_name' in locals():
                    tags['name'] = preserved_name
                # Rewrite old contact tags to contact:* tag form
                tags_rewrite = ['website', 'phone', 'email', 'facebook', 'instagram', 'youtube', 'pinterest', 'fax']

This line is too long as per the coding-style (116/100).

                for tr in tags_rewrite:

Variable name "tr" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    if tr in tags:
                        # Never overwrite already existing contact:* tags
                        if 'contact:' + tr in tags:
                            # We already have this contact:* tag so remove the simple contact tag
                            tags.pop(tr, None)
                        else:
                            # Rewrite simple contact tag to contact:* tag
                            tags['contact:' + tr] = tags.pop(tr, None)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                if row.get('poi_description') is not None and row.get('poi_description') != '':
                    tags['description'] = row.get('poi_description')
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # Write tags with yes/no value
                for k, v in POI_YESNO_TAGS.items():

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    if row.get(k) is not None and row.get(k) != '':
                        tags[v] = 'yes' if row.get(k) is True else 'no'
                for k, v in POI_EV_TAGS.items():

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    if row.get(k) is not None and row.get(k) != '':
                        if isinstance(row.get(k), float):
                            if not math.isnan(row.get(k)):
                                tags[v] = int(row.get(k))
                        else:
                            tags[v] = row.get(k)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # This is a new POI - will add fix me tag to the new items.
                if row.get('poi_new') is not None and row.get('poi_new') is True:
                    tags['fixme'] = 'verify import'
                # Remove unwanted addr:country from file output as we discussed in Issue #33
                tags.pop('addr:country', None)
                # tags['import'] = 'osm_poi_matchmaker'
                # Rendering tags to the XML file and JOSM magic link
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                josm_link = ''
                comment = '\nKey\t\t\t\tStatus\t\tNew value\t\tOSM value\n'
                for k, v in sorted(tags.items()):

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    xml_tags = etree.SubElement(main_data, 'tag', k=k, v='{}'.format(v))

Module 'lxml.etree' has no 'SubElement' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

The variable xml_tags seems to be unused.

                    josm_link = '{}|{}={}'.format(josm_link, k, v)
                    # Add original POI tags as comment
                    try:
                        if isinstance(v, str):
                            v = v.replace('-', '\-').replace('\n', '')

A suspicious escape sequence \- was found. Did you maybe forget to add an r prefix?

Variable name "v" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                        w = osm_live_tags[k]

Variable name "w" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                    except KeyError:
                        comment += "{:32} N\t\t'{}'\n".format(k, v)
                    else:
                        if isinstance(w, str):
                            w = w.replace('-', '\-').replace('\n', '')

A suspicious escape sequence \- was found. Did you maybe forget to add an r prefix?

Variable name "w" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                        comment += "{:32} {}\t\t'{}'\t\t\t'{}'\n".format(k, compare_strings(v, w), v, w)

This line is too long as per the coding-style (104/100).

                comment = etree.Comment(comment)

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                osm_xml_data.append(comment)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                # URL encode link and '--' in comment
                josm_link = quote(josm_link)
                josm_link = josm_link.replace('--', '%2D%2D')
                comment = etree.Comment(' JOSM magic link: {}?new_layer=false&objects={}&addtags={} '.format

This line is too long as per the coding-style (108/100).

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                                        ('http://localhost:8111/load_object', josm_object, josm_link))

The variable josm_object does not seem to be defined for all execution paths.

This line is too long as per the coding-style (102/100).

                osm_xml_data.append(comment)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                test_case = {k: row.get(k, None) for k in TESTCASE_GEN_KEYS}
                comment = etree.Comment("ˇ'original': '{t[original]}', 'postcode': '{t[poi_postcode]}', 'city': '{t[poi_city]}', 'street': '{t[poi_addr_street]}', 'housenumber': '{t[poi_addr_housenumber]}', 'conscriptionnumber': '{t[poi_conscriptionnumber]}'°".format(t=test_case))

This line is too long as per the coding-style (281/100).

Module 'lxml.etree' has no 'Comment' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.

                osm_xml_data.append(comment)
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
            try:
                osm_xml_data.append(xml_node_tags)

The variable xml_node_tags does not seem to be defined for all execution paths.

            except UnboundLocalError as e:

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.debug('Unbound local error extra node tags')
            try:
                osm_xml_data.append(main_data)
                # Next deafult OSM id is one more less for non existing objects
                default_osm_id -= 1
            except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

                logging.exception('Exception occurred')
    except ValueError as e:

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

        logging.error(e)
        logging.exception('Exception occurred')

    except Exception as e:

Catching very general exceptions such as Exception is usually not recommended.

Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

        logging.error(e)
        logging.exception('Exception occurred')

    return lxml.etree.tostring(osm_xml_data, pretty_print=True, xml_declaration=True, encoding="UTF-8")

This line is too long as per the coding-style (103/100).

Module 'lxml.etree' has no 'tostring' member, but source is unavailable. Consider adding this module to extension-pkg-whitelist if you want to perform analysis based on run-time introspection of living objects.