HydrologicEngineeringCenter / Vortex

data processing utilities
MIT License
28 stars 7 forks source link

Grid to point converter gives a time series of integers #11

Closed danhamill closed 4 years ago

danhamill commented 4 years ago

When I use the grid to point utility, I get a time series of integer values.

Here is shapefile: Willow_Creek.zip

Here is a DSS file with grids input_dss_grids.zip

Here is a file with the expected results Expected_Zonal_Stats.zip

Shouldn't the zonal statistics be floats?

tombrauer commented 4 years ago

Dan, can you send the dss grids again? I'm not sure what happened with them but the zip is empty. You can also send via the HEC dropbox or via MS Teams.

danhamill commented 4 years ago

I'll try again tomorrow. I've had other problems with uploading zip archives lately.

danhamill commented 4 years ago

How about this upload.

Input_Dss_Grids.zip

tombrauer commented 4 years ago

Good deal, I have the grids now. I wonder if there's a expiry on file exchanges...

I'll let you know what I find, once I get a chance to dig in.

tombrauer commented 4 years ago

I ran the grid-to-point-converter with your dataset. I spot checked a few values with you expected values spreadsheet and the values look really similar (screen cap below). Good to know.

image

I wasn't able to reproduce the integer time-series issue. Maybe you have a bad version of HEC-DSS?

danhamill commented 4 years ago

Hmm.  Maybe it is my version of dss.  I’ve been bouncing between 3.0.00.212 and 2.6.   I’ll check tomorrow.  

Your results appear correct to me.  I’ll report back.  Thanks for checking! nnk2193 notifications@github.com wrote: “I ran the grid-to-point-converter with your dataset. I spot checked a few values with you expected values spreadsheet and the values look really similar (screen cap below). Good to know.”

[image] “I wasn't able to reproduce the integer time-series issue. Maybe you have a bad version of HEC-DSS?”

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

tombrauer commented 4 years ago

@ktarbet can you get @danhamill a new build of HEC-DSSVue? I am working with v.3.0.1.125

danhamill commented 4 years ago

Confirmed the version 3.0.00.22 is a wonky version of DSS. When I review the grid to point results in dss 2.6, I am seeing an output similar to your demonstration.

tombrauer commented 4 years ago

Ok, good to know. Be careful with the 3.* DSSVue builds. They are all unofficial and are all broken in one way or another. I will say - I've been v.3.0.1.125 for about a month now, and haven't found any bugs 🤞

danhamill commented 4 years ago

I think the only reason 3.0.00.22 was on my radar was because you could directly import soil moisture data from snotel.

tombrauer commented 4 years ago

Snotel should be a plugin. If that's the case you can drag and drop it into any DSS build. image

danhamill commented 4 years ago

:shipit::shipit::shipit::shipit:

ktarbet commented 4 years ago

@ktarbet can you get @danhamill a new build of HEC-DSSVue? I am working with v.3.0.1.125

yes, on the way.

danhamill commented 4 years ago

@ktarbet Thank you, download received.

After downloading and extracting the new build of DSS, I am having the same issue. I found if i change the precision in tabulate, i see the additional digits. Where can I globally set the tabular precision? I cant seem to find any menus for default program settings.

ktarbet commented 4 years ago

I see the same issue, the number of decimals selected is not saved between tables. I've added this to our internal Issue tracker.

image

danhamill commented 4 years ago

That't the menu I found too. Is there a way to export the data to excel with the desired precision?

ktarbet commented 4 years ago

@danhamill The export should be including all decimal places in the floating point number. In Excel you can format to desired style.