Skip to content
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

[WIP] xy-grid extrusion along z #1965

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

matthewturk
Copy link
Member

This implements the first steps toward a data index that is defined by an xy grid of points with constant z. For instance, the particular use case is ARM data that is taken at multiple sweeps along the azimuth, at roughly onstant elevation, and always constant "range" (distance). This could be defined as a hexahedral mesh, but because we have a reduced dimensionality in xy, it can reduce memory considerably to define it with xy_grid and z_grid.

Also! I may have messed up my git branching! So there's the likely possibility that I'm including irrelevant changesets! But, when Nathan's PR goes in and I rebase, maybe that will go away?

There are numerous todo's, including:

  • finish implementation of extrusion base data object
  • finish fwidth
  • integrate into pixelizers the new extruded mesh pixelizer
  • implement cf_sweep frontend
  • Code passes flake8 checker
  • New features are documented, with docstrings and narrative docs
  • Adds a test for any bugs fixed. Adds tests for new features.

@matthewturk matthewturk added enhancement Making something better viz: 2D infrastructure Related to CI, versioning, websites, organizational issues, etc labels Aug 10, 2018
@matthewturk matthewturk added this to In progress in Weather data Mar 6, 2019
Base automatically changed from master to main January 20, 2021 15:27
@matthewturk matthewturk marked this pull request as draft May 18, 2021 19:34
@matthewturk
Copy link
Member Author

I'm converting this to draft until after 4.0 goes out so that it won't show up in my filters. This is a worthwhile change, but I need to update it and move forward with an updated version.

@matthewturk matthewturk self-assigned this Jan 21, 2022
@matthewturk
Copy link
Member Author

I do want to revisit this, albeit perhaps with a rebase, at some point in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Making something better infrastructure Related to CI, versioning, websites, organizational issues, etc viz: 2D
Projects
Weather data
  
In progress
Development

Successfully merging this pull request may close these issues.

None yet

2 participants