Cavewhere / cavewhere

3D Cave Mapping Software
https://cavewhere.com
GNU General Public License v3.0
34 stars 8 forks source link

Survex import not working #98

Closed ErinMLynch closed 8 years ago

ErinMLynch commented 8 years ago

Survex import is not working for either of the attached files. I click on Import -> svx, then select a file. The file is partially processed and I get a few warnings, but once I click "Import", the dialog closes and no data is added. The terminal window shows:

Creating temp files: "C:/Users/Erin/AppData/Local/Temp/CavewhereTmpProject-151f610762f.cw" Remove Task: cwRemoveImageTask(0x3f25ef0) TODO, implement this function!!! libpng warning: iCCP: known incorrect sRGB profile Couldn't show unknown page!!! This is a bug Z:\Documents\Projects\cavewhere\src\cwPageView.cpp 112 Cavern has crashed! "Survex 1.2.15\r\nCopyright \xB8 1990-2014 Olly Betts\r\nC:/ Users/Erin/AppData/Local/Temp/Cavewhere.Hp8916:13:6: warning: Closing prefix omitted from END\r\ncavern.exe: No survey data\r\n"

Note that both of my the attached files compile properly using Survex 1.2.24

Cheers, Erin

48H-L6-001-xiao.zip

vpicaver commented 8 years ago

Hey! So the attached data does work. I think you discovered that the survex dialog isn't very user friendly. If you click on the "enterance" and then assign that it's a trip, it should work. I need to disable the "Import" button until the user selects the correct option.

screen shot 2015-12-30 at 11 26 10 pm

ErinMLynch commented 8 years ago

Hmm... it is not intuitive to click on "entrance". Perhaps it should be selected by default? Cheers,Erin

On Thursday, 31 December 2015, 12:28, vpicaver <notifications@github.com> wrote:

Hey! So the attached data does work. I think you discovered that the survex dialog isn't very user friendly. If you click on the "enterance" and then assign that it's a trip, it should work. I need to disable the "Import" button until the user selects the correct option. — Reply to this email directly or view it on GitHub.

vpicaver commented 8 years ago

Yea, I agree, the dialog needs some work.

On Thu, Dec 31, 2015 at 2:03 AM, ErinMLynch notifications@github.com wrote:

Hmm... it is not intuitive to click on "entrance". Perhaps it should be selected by default? Cheers,Erin

On Thursday, 31 December 2015, 12:28, vpicaver notifications@github.com wrote:

Hey! So the attached data does work. I think you discovered that the survex dialog isn't very user friendly. If you click on the "enterance" and then assign that it's a trip, it should work. I need to disable the "Import" button until the user selects the correct option. —

Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/Cavewhere/cavewhere/issues/98#issuecomment-168137973.

vpicaver commented 8 years ago

You not the first person that has run into this issue.

Philip

On Thursday, December 31, 2015, Philip Schuchardt vpicaver@gmail.com wrote:

Yea, I agree, the dialog needs some work.

On Thu, Dec 31, 2015 at 2:03 AM, ErinMLynch <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

Hmm... it is not intuitive to click on "entrance". Perhaps it should be selected by default? Cheers,Erin

On Thursday, 31 December 2015, 12:28, vpicaver <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

Hey! So the attached data does work. I think you discovered that the survex dialog isn't very user friendly. If you click on the "enterance" and then assign that it's a trip, it should work. I need to disable the "Import" button until the user selects the correct option. —

Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/Cavewhere/cavewhere/issues/98#issuecomment-168137973 .

Phi|ip

Sent from Gmail Mobile

vpicaver commented 8 years ago

I've improved the dialog to disable the import button until you select something to import. The dialog should probably be rewritten in QML to support a more intutive design.