#73 Unable to save profile generated

[Tony Whitmore]

Having completed the calibration tests with a colorimeter, then unable to save the profile. Clicking the "Create Profile" button brings up an alert:

"The directory used for the output file is not valid. Please correct this condition and retry the operation."

The target directory exists and is writable, e.g. /home/tony/test.icc. The only output on the console is:

QDir::exists: Empty or null file name

[Hal Engel]

Logged In: YES
user_id=1052244
Originator: NO

Looking at the code and I found it. It turns out that I was not doing the right thing is there where no path included with the file name. IE.

/home/tony/test/icc

would have worked.

test.icc

when the current directory is /home/tony was not working. I fixed it and it tested OK on my machine. I have left in the the debugging statements just in case but I don't expect that you will have this problem again. I will remove the debugging statements once you confirm that it worked for you.

[Hal Engel]

Logged In: YES
user_id=1052244
Originator: NO

I have checked this fixed code into CVS. Please test it.

[Tony Whitmore]

Logged In: YES
user_id=633934
Originator: YES

This problem still occurs for me with the latest CVS, except there is no longer any output printed to the console! I have tried the following entries in the "Output Profile File" box:

test
test.icc
/home/tony/test.icc
/home/tony/test
/home/tony/Desktop/test.icc

All return the following error:
"The directory used for the output file is not valid. It either does not exist or is not a writable location. Please correct this condition and retry the operation."

[Hal Engel]

Logged In: YES
user_id=1052244
Originator: NO

OK I found another issue. I was testing this with the scanner profiler which was having the problem I wrote about earlier. The problem I fixed earlier would have affected the display profiler as well. So it needed to be fixed. The problem was that the error checking code was not hooked up correctly to also handle display profiles.

I have corrected the problem (I think since I have my machine configured to test another issue I can not test it right now). But since it was a simple change I expect it will work. The change is in CVS.

[Tony Whitmore]

Logged In: YES
user_id=633934
Originator: YES

Sorry, still getting the same error on latest CVS.

[Hal Engel]

Logged In: YES
user_id=1052244
Originator: NO

OK This time I think I really got it. I have tested a bunch of sceneries and everything seems to be working. There may still be some end cases that the code does not handle correctly but at this point I don't know what these may be since the code covers every mistake a user could make that I can think of and also should allow every valid case I can think of. I expect that this will do the right thing >99% of the time now. But of course please test it and make sure it is working for you.

[Hal Engel]

Logged In: YES
user_id=1052244
Originator: NO

The change is in CVS.

[Tony Whitmore]

Logged In: YES
user_id=633934
Originator: YES

Working great in current CVS! Thanks a million!