Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bgt timezone #234

Merged
merged 2 commits into from
Jan 28, 2018
Merged

Bgt timezone #234

merged 2 commits into from
Jan 28, 2018

Conversation

borrob
Copy link
Contributor

@borrob borrob commented Jan 21, 2018

Deeloplossing voor #233 (allen BGT).

De datum/tijdvelden worden nog steeds in de temp-tabellen ingeladen. Bij het overzetten naar de 'echte' tabellen wordt de tijdzone geïntroduceerd. Hierbij is aangenomen dat alles in 'Europe/Amsterdam' is.

@fsteggink
Copy link
Member

Sorry, ik had dit PR nog niet gezien toen ik afgelopen week met de BGT bezig ging, dus de code heeft nu een conflict. Ik heb wel gezien dat je aan het werk bent geweest voor de BAG, maar ik had niet door dat er ook een PR voor de BGT was.

@fsteggink fsteggink merged commit dacd53e into nlextract:master Jan 28, 2018
@fsteggink
Copy link
Member

Ik heb je fix gemerged. Nog niet getest, maar het ziet er goed uit. Ook moet ik nog aan de hand met hetzelfde script. In het kader van mijn eigen aanpassingen (zie o.a. 2598d90) heb ik veel aanpassingen gedaan, maar er treden wat fouten op wanner ik de hele dataset verwerk.

Het script robust-download.sh komt trouwens ook voor bij de BRK.

@borrob
Copy link
Contributor Author

borrob commented Jan 28, 2018

No worries. Ik zag dat je inderdaad flink bezig was geweest met het inladen van de BGT gegevens, dus ergens zag ik al aankomen dat het ging conflicteren. Succes met alle aanpassingen!

@borrob borrob deleted the bgt_timezone branch January 28, 2018 22:47
@justb4 justb4 added this to the Versie 1.4.0 milestone Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants