breedfides / airflow-etl

0 stars 2 forks source link

Implement radiation data Fetching/Clipping DAG #9

Closed brightemahpixida closed 8 months ago

brightemahpixida commented 10 months ago

Overview: This pull request adds a new DAG (fetch_cdc_radiation_DAG) to the ETL pipeline - the role of this DAG is centered on:

Changes Made:

gannebamm commented 10 months ago

@kuwet2k please take a look at the code and try to understand the changes. If in doubt, ask @brightemahpixida about it.

brightemahpixida commented 10 months ago

New Commit Update - ea30268:

Observations:

brightemahpixida commented 10 months ago

@kuwet2k @gannebamm this new commit can be used during the code review

brightemahpixida commented 10 months ago

New Commit Update - 4f71a60:

I switched the output format from Parquet to netCDF, indicating that all our clipped results will now be saved in the netCDF4 format.

gannebamm commented 9 months ago

@kuwet2k Please again take a look at the latest changes

gannebamm commented 9 months ago

@brightemahpixida I have started the review today. Is this code already deployed on the deNBI node?

brightemahpixida commented 9 months ago

@brightemahpixida I have started the review today. Is this code already deployed on the deNBI node?

Hi @gannebamm - yes this code has been deployed to the deNBI node; though right now the interface/version you are seeing right now when you visit the airflow UI is what's served on the main branch.

I could switch that to the latest version now so you could test it out

gannebamm commented 9 months ago

@brightemahpixida I have started the review today. Is this code already deployed on the deNBI node?

Hi @gannebamm ... I could switch that to the latest version now so you could test it out

That would be great. Please do this. It will help in testing the dags

brightemahpixida commented 9 months ago

@brightemahpixida I have started the review today. Is this code already deployed on the deNBI node?

Hi @gannebamm ... I could switch that to the latest version now so you could test it out

That would be great. Please do this. It will help in testing the dags

Hi @gannebamm - i've updated the DAGs on the airflow UI, you can check it out :)

Happy to help out if you encounter any problem or assistance with the test

brightemahpixida commented 9 months ago

👍