Skip to content

WIP: Prototype example of a cartobox...#1248

Closed
pelson wants to merge 1 commit into
SciTools:masterfrom
pelson:logo_box
Closed

WIP: Prototype example of a cartobox...#1248
pelson wants to merge 1 commit into
SciTools:masterfrom
pelson:logo_box

Conversation

@pelson

@pelson pelson commented Dec 26, 2018

Copy link
Copy Markdown
Member

Rationale

Adds a new example of next year's hottest Christmas toy... the cartobox. 🎄 ☃️

cartopy_cube_printout

Requires:

Implications

More clutter on everybody's desk.

import cartopy.crs as ccrs
from cartopy.examples.logo import add_cartopy_logo
import matplotlib.pyplot as plt
import matplotlib.patches

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

F401 'matplotlib.patches' imported but unused


def rect_healpix_tabs(ax):
tab_width = 2e6
tab_step = 2e6

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

F841 local variable 'tab_step' is assigned to but never used


paths = [
# Side.
mpath.Path([[2e7, 5e6], [2e7 + tab_width, 3e6], [2.2e7, -3e6], [2e7, -5e6]]),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E501 line too long (86 > 79 characters)

edgecolor='black', transform=ax.transData, clip_on=False, zorder=-1)
ax.add_collection(healpix_tabs)

def main():

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E302 expected 2 blank lines, found 1


def main():
fig = plt.figure(figsize=[12, 6])
ax = fig.add_subplot(1, 1, 1, projection=ccrs.RectangularHealpix(central_longitude=0, north_square=1, south_square=2))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E501 line too long (122 > 79 characters)

@greglucas

Copy link
Copy Markdown
Contributor

I'm not sure what this is demonstrating, so closing for now. Feel free to reopen if this is still desired.

@greglucas greglucas closed this Nov 14, 2021
@rcomer rcomer mentioned this pull request Jan 30, 2023
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