Urban-Meteorology-Reading / WRF-SUEWS

WRF-SUEWS coupling project
https://wrf-suews.readthedocs.org
MIT License
5 stars 2 forks source link

TODO: SUEWS fatal error needs to communicate with WRF fatal error #73

Closed hamidrezaomidvar closed 4 years ago

hamidrezaomidvar commented 4 years ago

It seems to me that when SUEWS fatal error happens, it does not communicate with WRF error module and although the run stops, in the cluster it shows the job still running. It might be the problem of Jasmin but good to double check! not an urgent problem to look into now

sunt05 commented 4 years ago

As I recall, we did have implemented this to trigger WRF fatal error if things went crazy: https://github.com/Urban-Meteorology-Reading/SUEWS/blob/47c08e5edea805ee976a9258f566190938225f9a/SUEWS-SourceCode/suews_ctrl_error.f95#L337-L344

These lines need to be activated when compiling by passing cpp flag for preprocessing.

zhenkunl commented 4 years ago

I agree with you

在 2020年5月15日,23:36,Ting Sun notifications@github.com 写道:

 As I recall, we did have implemented this to trigger WRF fatal error if things went crazy: https://github.com/Urban-Meteorology-Reading/SUEWS/blob/47c08e5edea805ee976a9258f566190938225f9a/SUEWS-SourceCode/suews_ctrl_error.f95#L337-L344

These lines need to be activated when compiling by passing cpp flag for preprocessing.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

hamidrezaomidvar commented 4 years ago

I see. Then it was definitely JASMIN problem. it has been very unstable recently because of the situation.