laravolt / indonesia

Package Laravel yang berisi data Provinsi, Kabupaten/Kota, dan Kecamatan/Desa di seluruh Indonesia.
MIT License
571 stars 200 forks source link

[usul] penambahan data latitude longitude #42

Closed lakuapik closed 4 years ago

lakuapik commented 4 years ago

Halo bung,

Saya tertarik dengan projek ini. Keren 👍

Kemarin saya ada projek yang pakai data-data ini, tapi disisi lain perlu ada data letak latitude longitude di bagian kota/kab (district). Saya cari-cari tidak ada, lalu saya coba generate sendiri waktu itu pakai api google geocoding.

Ini hasil filenya yang sudah ada latitude longitude: https://archive.org/download/administrative_indonesia_districts_with_lat_lng/administrative_indonesia_districts_with_lat_lng.csv

Kira-kira, saya ada usul nih. Data yg sekarang ditambahkan letak latitude longitudenya. Saat ini saya hanya ada yang district, kalau oke, nanti bisa di open issue biar teman-teman lain bisa membantu melengkapi.

Terima Kasih.

rizkyarlin commented 4 years ago

Ini keren sih mas @uyab, drpd yg make harus manggil API google geocoding setiap mau ambil latitude longitude. Ini juga bisa ngebantu programmer yg belum punya akses ke API Geocoding, mungkin karena gak punya Credit Card.

Untuk generatenya sepertinya bagus buat repo sendiri kali yah?

uyab commented 4 years ago

Sip, kita sudah punya kolom meta yang memang didesain untuk menyimpan data pendukung (yang bisa jadi setiap developer berbeda kebutuhannya). Saya sepakat untuk membuat repo terpisah, bisa di akun masing-masing (nanti dimention di readme) atau tetap di org laravolt.

Sampai sini ada tanggapan? Kalau oke, bisa langsung ekseskusi.

lakuapik commented 4 years ago

Mantap, saya setuju bung.

rizkyarlin commented 4 years ago

Di laravolt aja mas

uyab commented 4 years ago

Saya minta bantuan @lakuapik dan/atau @rizkyarlin ya untuk bikin MVP nya, nanti kalau sudah jadi tinggal ditransfer reponya. Sepertinya sudah pada paham kan teknisnya?

Paling nitip dibikin command untuk seed datanya, misal laravolt:indonesia:sync-coordinate. Bisa jadi nanti ada yang mau bikin sync-populasi, sync-luas-wilayah, dan lain-lain.

rizkyarlin commented 4 years ago

mas @lakuapik aja yg buat reponya, karena doi udah ada codingan untuk API Geocodingnya

lakuapik commented 4 years ago

bung @rizkyarlin saja gmn yang init repo dan boilerplatenya, karena saya blm tahu gimana standardnya laravolt. Nanti kalau sudah ada saya bisa PR.

uyab commented 4 years ago

Ada yang sudah bergerak kah? Btw saya sedang ngoprek https://developers.google.com/maps/documentation/geocoding/start, sepertinya saya lebih percaya data dari API Google 😅

Nanti kalau semua sudah dapat datanya, saya share data mentahnya.

cc @anandiamy

rizkyarlin commented 4 years ago

mas @lakuapik jg pakai itu kok mas @uyab

uyab commented 4 years ago

Wah iya bener sekali, saya ke-skip bagian google geocodingnya. Saya dari pagi generate koordinat, baru dapat 82%:

image

Saya sudah siapkan repo terpisah, https://github.com/laravolt/indonesia-coordinate, nanti data mentahnya diupload kesitu dan ditambahkan command:

  1. laravolt:indonesia:seed-coordinate untuk membaca data mentah lalu memasukkan ke database
  2. laravolt:Indonesia:sync-coordinate untuk memanggil API geocoding dan memperbarui data mentah

Ada tanggapan kah?

anandiamy commented 4 years ago

kenapa gak pakai repo ini aja ?

uyab commented 4 years ago

Hmm, pertanyaan bagus, kenapa ya dulu indonesia-logo dipisah 🤔

rizkyarlin commented 4 years ago

repo ini aja kali yah, saya dulu mikirnya bakal agak beda arahnya, tp dgn yg dilakukan mas uyab sepertinya oke aja untuk langsung di repo ini

uyab commented 4 years ago

updated di master , bisa coba jalankan ulang laravolt:indonesia:seed dan mungkin bantu random check memastikan koordinatnya benar

lakuapik commented 4 years ago

mantap nih, udah dibikin ya @uyab @rizkyarlin

tapi ini kita masih harus masukan api key dari google geocodenya ya? saya pikir nanti ada resource file untuk coordinatesnya dan pas di seed tinggal masukin data lat lng ke metanya.

uyab commented 4 years ago

@lakuapik

laravolt:indonesia:seed bakal langsung seed data koordinat dari CSV.

laravolt:indonesia:sync-coordinate baru butuh api key.

Cek https://github.com/laravolt/indonesia/blob/master/resources/csv/cities.csv

anandiamy commented 4 years ago

nah data koordinat nya masukin package ini aja sih mas

supaya gak usah ngejalanin laravolt:indonesia:sync-coordinate

uyab commented 4 years ago

Ini kan sudah masuk, coba cek branch master @anandiamy 😅