Skip to content

Restore early lake labels, add kind_detail for lakes and etc#2047

Merged
peitili merged 27 commits intomasterfrom
nvkelso/water-lake-labels-fix
Feb 1, 2022
Merged

Restore early lake labels, add kind_detail for lakes and etc#2047
peitili merged 27 commits intomasterfrom
nvkelso/water-lake-labels-fix

Conversation

@nvkelso
Copy link
Copy Markdown
Member

@nvkelso nvkelso commented Jan 20, 2022

Connects with #2003 and followup of #2010 to restore "lake" water labels at low zooms.

Also non-breaking change implementation of #984 for the v1.x series (which should stay open as a separate breaking change for v2.x series).

  • Update tests
  • Update docs

@peitili
Copy link
Copy Markdown
Contributor

peitili commented Jan 26, 2022

there is a linter violations, you might want to fix them using the pre-commit

@peitili
Copy link
Copy Markdown
Contributor

peitili commented Jan 26, 2022

I am not sure why but the integration tests always failed with context deadline exceed.

Comment thread integration-test/628-standardize-water-kinds.py Outdated
Copy link
Copy Markdown
Member Author

@nvkelso nvkelso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(moved to comment in PR review instead)

Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
Comment thread integration-test/628-standardize-water-kinds.py Outdated
@nvkelso nvkelso changed the title WIP: Restore early lake labels Restore early lake labels, add kind_detail for lakes and etc Jan 28, 2022
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
Comment thread integration-test/2047-restore-early-lake-labels.py Outdated
local['zoom'] = zoom
# this is to make the name `properties` visible in the queries.yaml's
# where clause
local['properties'] = props
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may impact build time, but generally makes things more consistent and obvious.

@peitili peitili force-pushed the nvkelso/water-lake-labels-fix branch from 0b54f17 to 912b486 Compare February 1, 2022 00:51
Copy link
Copy Markdown
Member Author

@nvkelso nvkelso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Final set of code, tests, and docs LGTM, including Q&A of a mini build.

@peitili this is ready to merge

@peitili peitili merged commit 215fd7a into master Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants