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

L3 product: Snow height, ice surface height and surface height #10

Open
BaptisteVandecrux opened this issue Oct 12, 2020 · 6 comments
Open
Labels
enhancement New feature or request L2 Issues with L2 data L3 Issues with L3 data

Comments

@BaptisteVandecrux
Copy link
Member

In the L3 version of the dataset, there should be:

  • SnowHeightm: Snow height above the ice surface. Never negative. Calculated as BoomHeightOverBareIcem - BoomHeightm except for accumulation sites (EGP and KAN_U) where it will need to be manually reset to 0 at the end of each melt season.
  • IceSurfaceHeightm: Height of the bare ice surface. Calculated as DepthPressureTransducerAtInstallationm- DepthPressureTransducerm. Needs to be adjusted every time a new ablation hose is installed.
  • SurfaceHeightm: Height of the snow/ice surface. Calculated as IceSurfaceHeight + SnowHeightm

Illustation:
image

Originally posted by @BaptisteVandecrux in #2 (comment)

@BaptisteVandecrux
Copy link
Member Author

Toolbox for processing/adjusting these heights:
https://github.com/GEUS-PROMICE/PROMICE-AWS-toolbox

@mankoff
Copy link
Member

mankoff commented Oct 14, 2020

These variables are now defined in the current version of the variables DB: https://github.com/GEUS-PROMICE/PROMICE-AWS-processing/blame/045e2ab972163ab2d2ebe7c223029ac9d311cb32/data/variables.csv#L52-L55

The code to populate them has not yet been added to this code, or called from the AWS-toolbox code linked above.

@mankoff mankoff added L2 Issues with L2 data L3 Issues with L3 data labels Oct 15, 2020
@PennyHow
Copy link
Member

Update: we have actually removed these from the variables look up table. I'm not sure when this happened.

But we would still like snow height, surface height and ice surface height calculations adding to our processing in the future.

@PennyHow PennyHow added the enhancement New feature or request label Apr 12, 2023
@BaptisteVandecrux
Copy link
Member Author

Now that I have moved forward with more advanced cleaning of z_pt_cor, z_boom and z_stake, I think we are getting closer to being able to provide these variables.

I think it is higher priority than "enhancement" because snow depth is a WMO variable and could be distributed to many services. Good quality snow depth would also help to derive SWE from surface height.

One of the requirements are a "station type" attribute (#217) and a smarter processing pipeline, because deriving additional variable will slow down the overall processing.

@PennyHow
Copy link
Member

If you have some scripts alredy for deriving snow height, ice surface height and surface height then please share them and we can look at how they can be more efficient, and also how we can fit them into, and improve, the pypromice processing pipeline

@BaptisteVandecrux BaptisteVandecrux changed the title L3 product: Snow height, ice surface height and surface height L4 product: Snow height, ice surface height and surface height May 27, 2024
@BaptisteVandecrux
Copy link
Member Author

BaptisteVandecrux commented May 27, 2024

Made a first implementation here: https://github.com/GEUS-Glaciology-and-Climate/pypromice/tree/surface-height-thermistor-depth-processing

Here are the results: https://github.com/GEUS-Glaciology-and-Climate/PROMICE-AWS-diagnostic/tree/main/figures/surface_heights

I call it level 4 because it uses quite heavy sequence of filters and tests to stitch the surface height as seen by the SR50s and the pressure transducer.

I can also only be run on merged l3 data: to create a continuous surface height, transmission should be aware of what was the surface height in the raw files. And if we start to bridge between the transmission and raw file processing, then we might as well run that specific processing after the merge. There are some other variables that could be processed after the merging.

It allows afterwards to derive the thermistors' depth.

@BaptisteVandecrux BaptisteVandecrux changed the title L4 product: Snow height, ice surface height and surface height L3 product: Snow height, ice surface height and surface height Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request L2 Issues with L2 data L3 Issues with L3 data
Projects
None yet
3 participants