UMEP-dev / UMEP

Urban Multi-scale Environmental Predictor
https://umep-docs.readthedocs.io/
GNU General Public License v3.0
62 stars 15 forks source link

SIGFPE: Floating-point exception - erroneous arithmetic operation (SUEWS SIMPLE) #60

Closed sunt05 closed 6 years ago

sunt05 commented 6 years ago

Original report by Wen Hui CUI (Bitbucket: wenhuiCC, GitHub: wenhuiCC).

The original report had attachments: HK_2018_SUEWS_60.txt, RunControl.nml


Hi there,

I prepared the input files using SUEWS simple in QGIS. When I run SUEWS.exe in cmd window, the message I got was shown below: problem 1 reported to issue 20180923.PNG

The input variables were measured by an eddy covariance system. And the meteorological input file and RunContral.nml were attached for your reference. There was no problem.txt created.

May I ask you guys some advices? Thanks very much!

Best regards, Wenhui

sunt05 commented 6 years ago

Original comment by Ting SUN (Bitbucket: sunt05, GitHub: sunt05).


Hi Wenhui, Thanks for reporting this issue.

Problem: If you can have a look at this page, you'll notice your chosen option actually used observed L_down to calculate Qn; while your observed L_down is -999, which might have caused some runtime issue.

Solution: Please change NetRadiationMethod to 0 or 3 and give it another try.

Ting

sunt05 commented 6 years ago

Original comment by Wen Hui CUI (Bitbucket: wenhuiCC, GitHub: wenhuiCC).


Hi Ting,

Thanks very much for your correction. Now it has been run successfully.

By the way, previously I setting the "NetRadiationMethod" to 1 was referred to this page. The description for No. 5 is " Net all-wave radiation. Required if NetRadiationMethod =1". Now I knew it should be 0 if I would like to use measured Rn directly. Do you think it will be better if revising the website description to prevent the confusion for other users?

Thanks!

Best Wenhui

sunt05 commented 6 years ago

Original comment by Ting SUN (Bitbucket: sunt05, GitHub: sunt05).


Thanks for reporting the issue in the SUEWS manual, which was just corrected.

Ting

sunt05 commented 6 years ago

Original comment by Wen Hui CUI (Bitbucket: wenhuiCC, GitHub: wenhuiCC).


Thank you!

sunt05 commented 6 years ago

Original changes by Ting SUN (Bitbucket: sunt05, GitHub: sunt05).


changed state from "new" to "resolved"