Korjaukset #132
Open
Korjaukset #132
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Päivitetty kohdetyyppien määritys vastaamaan määritelmää:
1. Kohdetyyppien määritys (determine_kohdetyyppi):
- Korjattu asuinkiinteistön tunnistaminen määritelmän mukaiseksi:
* Rakennusluokka 0110-0211
* Käyttötarkoitus 011-041
* Huoneistomäärä > 0
* On asukas
- Selkeytetty HAPA/BIOHAPA logiikkaa:
* BIOHAPA: Vain jos on BIOHAPA-kohde JA ei biojätteen erilliskeräysalueella
* HAPA: Vain jos on HAPA-kohde JA ei ole asuinkiinteistö
Etäisyys max 300m
Yhden asunnon kohteet (get_or_create_single_asunto_kohteet):
Käsittelee oikeat rakennustyypit:
Omakotitalot (0111/011)
Paritalot (0110/012)
Käsittelee vain kiinteistöt joilla on yksi rakennus
Huomioi myös talot joissa ei ole asukkaita
Muut kohteet (get_or_create_multiple_and_uninhabited_kohteet):
Käsittelee kaikki jäljellä olevat rakennukset
Yhdistämiskriteerit:
Sama omistaja TAI asukas
Sama osoite
Etäisyys max 300m
Erityiskäsittelyt:
Asunto-osakeyhtiöiden rakennukset yhdistetään aina
Muut rakennukset käsitellään omistajittain
Apurakennukset liitetään päärakennuksiin
Apurakennusten käsittely (add_auxiliary_buildings_materialized):
Tunnistaa oikeat rakennustyypit:
Saunat (1910)
Talousrakennukset (1911)
Liittämiskriteerit:
Sama omistaja/asukas
Sama osoite
Etäisyys max 300m
Rakennusten klusterointi (_cluster_rakennustiedot):
Yhdistämiskriteerit:
Sama omistaja TAI asukas
Sama osoite
Etäisyys max 300m
Erityiskäsittelyt:
Asunto-osakeyhtiöt käsitellään yhtenä kokonaisuutena
Muut rakennukset jaetaan omistajien mukaan
The address number matching logic has been simplified to only consider the base number when grouping buildings into a kohde. This fixes issues where buildings with the same base number but different suffixes were incorrectly split into separate kohde entries. Examples of now correctly matched addresses: - Laurilankatu 14a, 14b, 14c - Laurilankatu 14 a, 14 b, 14 c - Laurilankatu 14 a 1, 14 b 2 - Laurilankatu 14 a 1, 14 a 2 Changes: - Add _extract_base_number helper function to extract the first number from address - Simplify _find_by_address to use only the base number for matching - Remove complex suffix handling logic - Use ILIKE operator for flexible address number matching
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Korjauksia kohteiden muodostamiseen ja tietokantaoikeuksiin
Muutokset
Kohteiden muodostaminen
Tietokanta
Dokumentaatio
Testaus
Vaikutukset