-
Notifications
You must be signed in to change notification settings - Fork 19
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
add the airports workflow #231
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
I have the following comments:
Originally, the code keeps only medium and large airports and make the weighting like this (Please note that the last line must be equal to 2 and not 3:
I suggest, that we keep only medium and large airports, as we originally did. |
I did not intend to keep the small one, that was commented out by mistake. I changed the sizing factor however to 3. Judging by a few checks in 3 countries, the large airports are usually much larger than the medium ones. I can set that as a config parameter to change accordingly |
Closes # (if applicable).
Changes proposed in this Pull Request
Checklist
envs/environment.yaml
andenvs/environment.docs.yaml
.config.default.yaml
,config.tutorial.yaml
, andtest/config.test1.yaml
.doc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes, including reference to the requested PR.