@hebcal/geo-sqlite 4.4.1 → 4.4.2

package/dist/index.js

@@ -1,4 +1,4 @@
-/*! @hebcal/geo-sqlite v4.4.1 */
+/*! @hebcal/geo-sqlite v4.4.2 */
 'use strict';
 
 Object.defineProperty(exports, '__esModule', { value: true });
@@ -29,6 +29,7 @@ const GEONAME_SQL = `SELECT
   a.asciiname as admin1,
   g.latitude as latitude,
   g.longitude as longitude,
+  g.population as population,
   g.timezone as timezone
 FROM geoname g
 LEFT JOIN country c on g.country = c.iso
@@ -44,27 +45,27 @@ const GEONAME_ALL_SQL = `SELECT
   a.asciiname as admin1,
   g.latitude as latitude,
   g.longitude as longitude,
+  g.population as population,
   g.timezone as timezone
 FROM geoname g
 LEFT JOIN country c on g.country = c.iso
 LEFT JOIN admin1 a on g.country||'.'||g.admin1 = a.key
 `;
-const ZIPCODE_SQL = `SELECT CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving
+const ZIPCODE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary WHERE ZipCode = ?`;
-const ZIPCODE_ALL_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving
+const ZIPCODE_ALL_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary`;
 const ZIP_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary
 WHERE ZipCode LIKE ?
 ORDER BY Population DESC
 LIMIT 10`;
-const ZIP_FULLTEXT_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
-FROM ZIPCodes_CityFullText
+const ZIP_FULLTEXT_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,rank
+FROM ZIPCodes_CityFullText5
 WHERE CityMixedCase MATCH ?
 ORDER BY Population DESC
 LIMIT 15`;
-const GEONAME_COMPLETE_SQL = `SELECT geonameid, asciiname, admin1, country,
-population, latitude, longitude, timezone,
+const GEONAME_COMPLETE_SQL = `SELECT geonameid, longname,
 ((sqrt(population)/40) + (-30 * rank)) as myrank
 FROM geoname_fulltext
 WHERE geoname_fulltext MATCH ?
@@ -226,6 +227,7 @@ class GeoDb {
     location.stateName = stateNames[location.state];
     location.geo = 'zip';
     location.zip = zip;
+    location.population = result.Population;
     return location;
   }
   /**
@@ -263,13 +265,9 @@ class GeoDb {
     const admin1 = result.admin1 || '';
     const cityDescr = core.Location.geonameCityDescr(result.name, admin1, country);
     const location = new core.Location(result.latitude, result.longitude, result.cc == 'IL', result.timezone, cityDescr, result.cc, geonameid);
-
-    if (result.asciiname !== result.name) {
-      location.asciiname = result.asciiname;
-    }
-
     location.geo = 'geoname';
     location.geonameid = geonameid;
+    location.asciiname = result.asciiname;
 
     if (admin1) {
       location.admin1 = admin1;
@@ -279,6 +277,10 @@ class GeoDb {
       location.jersualem = true;
     }
 
+    if (result.population) {
+      location.population = result.population;
+    }
+
     return location;
   }
   /**
@@ -358,23 +360,30 @@ class GeoDb {
       qraw = qraw.replace(/\"/g, '""');
       const geoRows = this.geonamesCompStmt.all(`{longname} : "${qraw}" *`);
       const geoMatches = geoRows.map(res => {
-        const country = res.country || '';
-        const admin1 = res.admin1 || '';
+        const loc = this.lookupGeoname(res.geonameid);
+        const country = this.countryNames.get(loc.getCountryCode()) || '';
+        const admin1 = loc.admin1 || '';
         const rank = Math.trunc(res.myrank * 100.0) / 100.0;
         const obj = {
           id: res.geonameid,
-          value: core.Location.geonameCityDescr(res.asciiname, admin1, country),
-          asciiname: res.asciiname,
+          value: loc.name,
           admin1,
           country,
-          latitude: res.latitude,
-          longitude: res.longitude,
-          timezone: res.timezone,
-          population: res.population,
+          latitude: loc.latitude,
+          longitude: loc.longitude,
+          timezone: loc.getTzid(),
           geo: 'geoname',
           rank: rank
         };
 
+        if (loc.population) {
+          obj.population = loc.population;
+        }
+
+        if (loc.asciiname) {
+          obj.asciiname = loc.asciiname;
+        }
+
         if (country) {
           obj.country = country;
         }
@@ -390,8 +399,26 @@ class GeoDb {
         this.zipFulltextCompStmt = this.zipsDb.prepare(ZIP_FULLTEXT_COMPLETE_SQL);
       }
 
-      const zipRows = this.zipFulltextCompStmt.all(`"${qraw}*"`);
-      const zipMatches = zipRows.map(GeoDb.zipResultToObj);
+      const zipRows = this.zipFulltextCompStmt.all(`"${qraw}" *`);
+      const zipMatches = zipRows.map(res => {
+        const zipCode = res.ZipCode;
+        const loc = this.lookupZip(zipCode);
+        const myrank = Math.sqrt(loc.population) / 40 + -30 * res.rank;
+        const obj = {
+          id: zipCode,
+          value: loc.getName(),
+          admin1: loc.admin1,
+          asciiname: loc.getShortName(),
+          country: 'United States',
+          latitude: loc.latitude,
+          longitude: loc.longitude,
+          timezone: loc.getTzid(),
+          population: loc.population,
+          geo: 'zip',
+          rank: myrank
+        };
+        return obj;
+      });
       const map = new Map();
 
       for (const obj of zipMatches) {
@@ -409,7 +436,7 @@ class GeoDb {
       }
 
       const values = Array.from(map.values());
-      values.sort((a, b) => b.population - a.population);
+      values.sort((a, b) => b.rank - a.rank);
       const topN = values.slice(0, 10);
 
       if (!latlong) {
@@ -601,17 +628,12 @@ async function buildGeonamesSqlite(opts) {
     `);
   doSql(logger, db, `update admin1 set name='',asciiname='' where key like 'PS.%';`, `update country set country = '' where iso = 'PS';`, `delete from geoname where geonameid = 7303419;`);
   doSql(logger, db, `DROP TABLE IF EXISTS geoname_fulltext`, `CREATE VIRTUAL TABLE geoname_fulltext
-      USING fts5(geonameid,
-      longname,
-      asciiname, admin1, country,
-      population, latitude, longitude, timezone
-      );
+      USING fts5(geonameid, longname, population);
     `, `DROP TABLE IF EXISTS geoname_non_ascii`, `CREATE TABLE geoname_non_ascii AS
       SELECT geonameid FROM geoname WHERE asciiname <> name`, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||a.asciiname||', '||c.Country,
-      g.asciiname, a.asciiname, c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a, country c
       WHERE g.country = c.ISO
       AND g.country <> 'US'
@@ -619,24 +641,21 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||a.asciiname||', USA',
-      g.asciiname, a.asciiname, 'United States',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a
       WHERE g.country = 'US'
       AND g.country||'.'||g.admin1 = a.key
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||c.Country,
-      g.asciiname, '', c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, country c
       WHERE g.country = c.ISO
       AND (g.admin1 = '' OR g.admin1 = '00')
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.name||', '||a.name||', '||c.Country,
-      g.name, a.name, c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname_non_ascii gna, geoname g, admin1 a, country c
       WHERE gna.geonameid = g.geonameid
       AND g.country = c.ISO
@@ -644,8 +663,7 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       alt.name||', ישראל',
-      alt.name, '', 'ישראל',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, altnames alt
       WHERE g.country = 'IL'
       AND alt.isolanguage = 'he'
@@ -653,8 +671,7 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       alt.name||', '||a1.asciiname||', Israel',
-      alt.name, a1.asciiname, 'Israel',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a1, altnames alt
       WHERE g.country = 'IL'
       AND alt.isolanguage = 'en'

package/dist/index.mjs

@@ -1,4 +1,4 @@
-/*! @hebcal/geo-sqlite v4.4.1 */
+/*! @hebcal/geo-sqlite v4.4.2 */
 import Database from 'better-sqlite3';
 import { Location, Locale } from '@hebcal/core';
 import '@hebcal/cities';
@@ -17,6 +17,7 @@ const GEONAME_SQL = `SELECT
   a.asciiname as admin1,
   g.latitude as latitude,
   g.longitude as longitude,
+  g.population as population,
   g.timezone as timezone
 FROM geoname g
 LEFT JOIN country c on g.country = c.iso
@@ -32,27 +33,27 @@ const GEONAME_ALL_SQL = `SELECT
   a.asciiname as admin1,
   g.latitude as latitude,
   g.longitude as longitude,
+  g.population as population,
   g.timezone as timezone
 FROM geoname g
 LEFT JOIN country c on g.country = c.iso
 LEFT JOIN admin1 a on g.country||'.'||g.admin1 = a.key
 `;
-const ZIPCODE_SQL = `SELECT CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving
+const ZIPCODE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary WHERE ZipCode = ?`;
-const ZIPCODE_ALL_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving
+const ZIPCODE_ALL_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary`;
 const ZIP_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
 FROM ZIPCodes_Primary
 WHERE ZipCode LIKE ?
 ORDER BY Population DESC
 LIMIT 10`;
-const ZIP_FULLTEXT_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population
-FROM ZIPCodes_CityFullText
+const ZIP_FULLTEXT_COMPLETE_SQL = `SELECT ZipCode,CityMixedCase,rank
+FROM ZIPCodes_CityFullText5
 WHERE CityMixedCase MATCH ?
 ORDER BY Population DESC
 LIMIT 15`;
-const GEONAME_COMPLETE_SQL = `SELECT geonameid, asciiname, admin1, country,
-population, latitude, longitude, timezone,
+const GEONAME_COMPLETE_SQL = `SELECT geonameid, longname,
 ((sqrt(population)/40) + (-30 * rank)) as myrank
 FROM geoname_fulltext
 WHERE geoname_fulltext MATCH ?
@@ -214,6 +215,7 @@ class GeoDb {
     location.stateName = stateNames[location.state];
     location.geo = 'zip';
     location.zip = zip;
+    location.population = result.Population;
     return location;
   }
   /**
@@ -251,13 +253,9 @@ class GeoDb {
     const admin1 = result.admin1 || '';
     const cityDescr = Location.geonameCityDescr(result.name, admin1, country);
     const location = new Location(result.latitude, result.longitude, result.cc == 'IL', result.timezone, cityDescr, result.cc, geonameid);
-
-    if (result.asciiname !== result.name) {
-      location.asciiname = result.asciiname;
-    }
-
     location.geo = 'geoname';
     location.geonameid = geonameid;
+    location.asciiname = result.asciiname;
 
     if (admin1) {
       location.admin1 = admin1;
@@ -267,6 +265,10 @@ class GeoDb {
       location.jersualem = true;
     }
 
+    if (result.population) {
+      location.population = result.population;
+    }
+
     return location;
   }
   /**
@@ -346,23 +348,30 @@ class GeoDb {
       qraw = qraw.replace(/\"/g, '""');
       const geoRows = this.geonamesCompStmt.all(`{longname} : "${qraw}" *`);
       const geoMatches = geoRows.map(res => {
-        const country = res.country || '';
-        const admin1 = res.admin1 || '';
+        const loc = this.lookupGeoname(res.geonameid);
+        const country = this.countryNames.get(loc.getCountryCode()) || '';
+        const admin1 = loc.admin1 || '';
         const rank = Math.trunc(res.myrank * 100.0) / 100.0;
         const obj = {
           id: res.geonameid,
-          value: Location.geonameCityDescr(res.asciiname, admin1, country),
-          asciiname: res.asciiname,
+          value: loc.name,
           admin1,
           country,
-          latitude: res.latitude,
-          longitude: res.longitude,
-          timezone: res.timezone,
-          population: res.population,
+          latitude: loc.latitude,
+          longitude: loc.longitude,
+          timezone: loc.getTzid(),
           geo: 'geoname',
           rank: rank
         };
 
+        if (loc.population) {
+          obj.population = loc.population;
+        }
+
+        if (loc.asciiname) {
+          obj.asciiname = loc.asciiname;
+        }
+
         if (country) {
           obj.country = country;
         }
@@ -378,8 +387,26 @@ class GeoDb {
         this.zipFulltextCompStmt = this.zipsDb.prepare(ZIP_FULLTEXT_COMPLETE_SQL);
       }
 
-      const zipRows = this.zipFulltextCompStmt.all(`"${qraw}*"`);
-      const zipMatches = zipRows.map(GeoDb.zipResultToObj);
+      const zipRows = this.zipFulltextCompStmt.all(`"${qraw}" *`);
+      const zipMatches = zipRows.map(res => {
+        const zipCode = res.ZipCode;
+        const loc = this.lookupZip(zipCode);
+        const myrank = Math.sqrt(loc.population) / 40 + -30 * res.rank;
+        const obj = {
+          id: zipCode,
+          value: loc.getName(),
+          admin1: loc.admin1,
+          asciiname: loc.getShortName(),
+          country: 'United States',
+          latitude: loc.latitude,
+          longitude: loc.longitude,
+          timezone: loc.getTzid(),
+          population: loc.population,
+          geo: 'zip',
+          rank: myrank
+        };
+        return obj;
+      });
       const map = new Map();
 
       for (const obj of zipMatches) {
@@ -397,7 +424,7 @@ class GeoDb {
       }
 
       const values = Array.from(map.values());
-      values.sort((a, b) => b.population - a.population);
+      values.sort((a, b) => b.rank - a.rank);
       const topN = values.slice(0, 10);
 
       if (!latlong) {
@@ -589,17 +616,12 @@ async function buildGeonamesSqlite(opts) {
     `);
   doSql(logger, db, `update admin1 set name='',asciiname='' where key like 'PS.%';`, `update country set country = '' where iso = 'PS';`, `delete from geoname where geonameid = 7303419;`);
   doSql(logger, db, `DROP TABLE IF EXISTS geoname_fulltext`, `CREATE VIRTUAL TABLE geoname_fulltext
-      USING fts5(geonameid,
-      longname,
-      asciiname, admin1, country,
-      population, latitude, longitude, timezone
-      );
+      USING fts5(geonameid, longname, population);
     `, `DROP TABLE IF EXISTS geoname_non_ascii`, `CREATE TABLE geoname_non_ascii AS
       SELECT geonameid FROM geoname WHERE asciiname <> name`, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||a.asciiname||', '||c.Country,
-      g.asciiname, a.asciiname, c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a, country c
       WHERE g.country = c.ISO
       AND g.country <> 'US'
@@ -607,24 +629,21 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||a.asciiname||', USA',
-      g.asciiname, a.asciiname, 'United States',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a
       WHERE g.country = 'US'
       AND g.country||'.'||g.admin1 = a.key
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.asciiname||', '||c.Country,
-      g.asciiname, '', c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, country c
       WHERE g.country = c.ISO
       AND (g.admin1 = '' OR g.admin1 = '00')
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       g.name||', '||a.name||', '||c.Country,
-      g.name, a.name, c.Country,
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname_non_ascii gna, geoname g, admin1 a, country c
       WHERE gna.geonameid = g.geonameid
       AND g.country = c.ISO
@@ -632,8 +651,7 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       alt.name||', ישראל',
-      alt.name, '', 'ישראל',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, altnames alt
       WHERE g.country = 'IL'
       AND alt.isolanguage = 'he'
@@ -641,8 +659,7 @@ async function buildGeonamesSqlite(opts) {
       `, `INSERT INTO geoname_fulltext
       SELECT g.geonameid,
       alt.name||', '||a1.asciiname||', Israel',
-      alt.name, a1.asciiname, 'Israel',
-      g.population, g.latitude, g.longitude, g.timezone
+      g.population
       FROM geoname g, admin1 a1, altnames alt
       WHERE g.country = 'IL'
       AND alt.isolanguage = 'en'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hebcal/geo-sqlite",
-  "version": "4.4.1",
+  "version": "4.4.2",
   "author": "Michael J. Radwin (https://github.com/mjradwin)",
   "keywords": [
     "hebcal"

package/zips-dummy.sql

@@ -10,4 +10,7 @@ CREATE TABLE ZIPCodes_Primary (
 );
 
 CREATE VIRTUAL TABLE ZIPCodes_CityFullText
-USING fts3(ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population);
+USING fts4(ZipCode,CityMixedCase,State,Latitude,Longitude,TimeZone,DayLightSaving,Population);
+
+CREATE VIRTUAL TABLE ZIPCodes_CityFullText5
+USING fts5(ZipCode,CityMixedCase,Population);