Skip to content

Add ObsTool to AccordDaTools#20

Open
JanaSanchez wants to merge 18 commits into
ACCORD-NWP:developfrom
JanaSanchez:FeatureAdd/ObsTool
Open

Add ObsTool to AccordDaTools#20
JanaSanchez wants to merge 18 commits into
ACCORD-NWP:developfrom
JanaSanchez:FeatureAdd/ObsTool

Conversation

@JanaSanchez

Copy link
Copy Markdown

No description provided.

@ewhelan
ewhelan self-requested a review February 1, 2022 10:10
@ewhelan ewhelan self-assigned this Feb 1, 2022
@ewhelan ewhelan added the enhancement New feature or request label Feb 1, 2022
@ewhelan

ewhelan commented Feb 1, 2022

Copy link
Copy Markdown
Collaborator

Thanks @JanaSanchez . Let's work on this together ...

@ewhelan

ewhelan commented Feb 1, 2022

Copy link
Copy Markdown
Collaborator

Hola @JanaSanchez

I have made some updates to the data preparation steps. Could you try the following on tornado:

cd /path/to/AccordDaTools
git pull
mkdir build &&  cd build
cmake .. -DCMAKE_INSTALL_PREFIX=$HOME/js_da_tools && make && make install
export PATH=$HOME/js_da_tools/bin:$PATH
cd /path/to/ccma_mon_data
1prepare_obstool -i input -s 2020123112 -e 2020123115 ## pick your own dates

I used the following SQL for SYNOP: odbsql -T -q 'SELECT rad2deg(lat),rad2deg(lon),an_depar,fg_depar,obsvalue FROM hdr,body,desc where obstype=1 AND an_depar IS NOT NULL' > /path/to/ccma_mon_data/input/ccma_mon_2020123112

@ewhelan ewhelan changed the title This is a documentation to use ObsTool Add ObsTool to AccordDaTools Feb 1, 2022

@ewhelan ewhelan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I have been very slow with this. Apologies. I would like to go through the steps you have documented before approving.

I will be in touch. Hopefully before the WW!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants