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

Experiment: How HDFReferenceRecipe could look as a Beam Pipeline. #337

Closed
wants to merge 2 commits into from

Conversation

alxmrs
Copy link
Contributor

@alxmrs alxmrs commented Apr 12, 2022

This is a prototype for using Apache Beam for the internal data model of Pangeo Forge Recipes. Here, I demo how HDFReferenceRecipe could be structured into modular components via composite Beam transforms.

xref: #256

This is a prototype for using Apache Beam for the internal (and external?) data model of Pangeo Forge Recipes. Here, I demo how HDFReferenceRecipe could be structured into modular components via composite Beam transforms.

xref: pangeo-forge#256
This is a prototype for using Apache Beam for the internal (and external?) data model of Pangeo Forge Recipes. Here, I demo how HDFReferenceRecipe could be structured into modular components via composite Beam transforms.

xref: pangeo-forge#256
@martindurant
Copy link
Contributor

Honestly, I find this pretty hard to read :|
Is it reasonable to set some time aside in the regular meeting to describe what's involved here?

Comment on lines +213 to +214
def to_beam(self):
return OpenPattern(self.file_pattern) | FileNames() | ScanFiles(self) | WriteZarrReference(self)
Copy link
Contributor

Choose a reason for hiding this comment

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

@martindurant - this is the part that is easy to read! 🤩

@rabernat
Copy link
Contributor

Beam programming takes some getting used to, but once it clicks, it seems super elegant to me.

Is it reasonable to set some time aside in the regular meeting to describe what's involved here?

Alex, Charles and I met today. We can discuss more at the next coordination meeting. We need to evaluate the pros and cons.

@cisaacstern
Copy link
Member

Thanks for this experiment! Closing since we now have all this (in some form) in 0.10.0 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants