kdknigga / aeroinfo Goto Github PK

Ditch random temp directory names for something more meaningful which will allow the importer to make helpful assumptions.

The README references this link as an example application:
https://api.aeronautical.info/dev/?airport=ORD&include=demographic&include=geographic&include=ownership&include=runways

However, the webpage is broken and displays this message: {"message": "Internal server error"}

To resolve this issue, recommend removing example application link from README or preferably fix the example application.

Based on what's in APT.txt, create all of the columns in the runways table and update the parser/importer to populate the columns.

For example, perhaps Base.metadata.create_all(Engine) isn't required for every script run?

Requires #1

download previous and current and compare apt_rf.txt

Based on what's in APT.txt, create all of the columns in the airports table and update the parser/importer to populate the columns.

...and probably also comments.

I am potentially interested in using this repository but there is no formal LICENSE or terms of use. I can't and won't use the repository without a license.

To resolve this issue, I recommend adding a LICENSE file and assigning a permissive open source license, such as BSD-2 or MIT.

This is a good resource in helping to select a license: https://choosealicense.com/

Requires #2

Come up with a way to manage database schema changes.

https://pypi.org/project/alembic/ ?

Break database.py into smaller parts in their own directory

UserWarning: The psycopg2 wheel package will be renamed from release 2.8; in order to keep installing from binary please use "pip install psycopg2-binary" instead. For details see: http://initd.org/psycopg/docs/install.html#binary-install-from-pypi.

Look into struct.Struct to unpack the fixed width data in import.py