Skip to content

feat(maps): replace legacy deckgl and mapbox plugins with new deckgl …#38730

Closed
birkskyum wants to merge 1 commit into
apache:masterfrom
birkskyum:squash-deckgl-maplibre
Closed

feat(maps): replace legacy deckgl and mapbox plugins with new deckgl …#38730
birkskyum wants to merge 1 commit into
apache:masterfrom
birkskyum:squash-deckgl-maplibre

Conversation

@birkskyum
Copy link
Copy Markdown
Contributor

…preset and point cluster map

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@github-actions github-actions Bot added risk:db-migration PRs that require a DB migration doc Namespace | Anything related to documentation plugins dependencies:npm github_actions Pull requests that update GitHub Actions code packages labels Mar 18, 2026
@birkskyum birkskyum force-pushed the squash-deckgl-maplibre branch from 89d8b06 to 4ae07e6 Compare March 18, 2026 18:18
@github-actions github-actions Bot removed the doc Namespace | Anything related to documentation label Mar 18, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 47.05882% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.44%. Comparing base (7909095) to head (4ae07e6).
⚠️ Report is 15 commits behind head on master.

Files with missing lines Patch % Lines
superset/viz.py 47.05% 9 Missing ⚠️

❌ Your project status has failed because the head coverage (99.92%) is below the target coverage (100.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #38730      +/-   ##
==========================================
- Coverage   65.01%   64.44%   -0.58%     
==========================================
  Files        1819     2537     +718     
  Lines       72626   130424   +57798     
  Branches    23196    30000    +6804     
==========================================
+ Hits        47218    84051   +36833     
- Misses      25408    44913   +19505     
- Partials        0     1460    +1460     
Flag Coverage Δ
hive 40.58% <47.05%> (?)
mysql 61.59% <47.05%> (?)
postgres 61.65% <47.05%> (?)
presto 40.60% <47.05%> (?)
python 63.26% <47.05%> (?)
sqlite 61.29% <47.05%> (?)
superset-extensions-cli 90.82% <ø> (?)
unit 100.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@birkskyum birkskyum closed this Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies:npm github_actions Pull requests that update GitHub Actions code packages plugins risk:db-migration PRs that require a DB migration size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant