Re: [hugin-ptx] Re: Optimizer tab and X,Y,Z parameters

2010-03-23 Thread Felix Hagemann 
On 17 March 2010 00:34, Bruno Postle  wrote:
>> The attached patch implements a "Everything (not x,y,z)" preset, which
>> does the job for me.
>
> Thanks, I committed it, but I fear that this preset menu is already very
> confusing.

Thanks for committing. I agree that the preset menu and probably the
whole Optimizer tab is confusing to inexperienced users.  I've been
trying to come up with a way to improve usability but I can't think of
a scheme that wouldn't make my own workflow cumbersome or even
impossible.

One approach would be to make the user decide early on in the process
whether he's working on a (i) linear, flat, mosaic (or whatever it's
called) panorama involving translation parameters or a (ii)
traditional one. For (i) automatically do optimization including
translation params and for (ii) activate them only for shots somehow
"tagged" as nadir shots.

But probably that's a usability nightmare as well and I'm pretty sure
that there will be people finding very valid and creative use cases
that involve both types in one project. Maybe we just have to accept
that hugin is an extremely powerful tool which can only lose part of
it's power if it's made "easy".

Felix

-- 
You received this message because you are subscribed to the Google Groups 
"hugin and other free panoramic software" group.
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to 
hugin-ptx+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptx

To unsubscribe from this group, send email to 
hugin-ptx+unsubscribegooglegroups.com or reply to this email with the words 
"REMOVE ME" as the subject.

Re: [hugin-ptx] Re: Optimizer tab and X,Y,Z parameters

2010-03-16 Thread Bruno Postle 

On Thu 11-Mar-2010 at 23:20 +0100, Felix Hagemann wrote:

On 10 March 2010 23:42, Felix Hagemann  wrote:

What would others think of a new preset like "Everything (without
XYZ)" to make things a tad easier? Or maybe even add a new check box
which toggles XYZ for all presets?


The attached patch implements a "Everything (not x,y,z)" preset, which
does the job for me.


Thanks, I committed it, but I fear that this preset menu is already 
very confusing.


--
Bruno

--
You received this message because you are subscribed to the Google Groups "hugin and 
other free panoramic software" group.
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to 
hugin-ptx+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptx

[hugin-ptx] Re: Optimizer tab and X,Y,Z parameters

2010-03-11 Thread Felix Hagemann 
On 10 March 2010 23:42, Felix Hagemann  wrote:
> What would others think of a new preset like "Everything (without
> XYZ)" to make things a tad easier? Or maybe even add a new check box
> which toggles XYZ for all presets?

The attached patch implements a "Everything (not x,y,z)" preset, which
does the job for me.

Felix

-- 
You received this message because you are subscribed to the Google Groups 
"hugin and other free panoramic software" group.
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to 
hugin-ptx+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptxIndex: src/hugin1/hugin/xrc/optimize_panel.xrc
===
--- src/hugin1/hugin/xrc/optimize_panel.xrc	(revision 5059)
+++ src/hugin1/hugin/xrc/optimize_panel.xrc	(working copy)
@@ -44,6 +44,7 @@
   Positions, Translation and Barrel (y,p,r,x,y,z,b)
   Positions, View and Barrel (y,p,r,v,b)
   Positions, Translation, View and Barrel (y,p,r,x,y,z,v,b)
+  Everything (not x,y,z)
   Everything
   the Custom parameters below
 
Index: src/hugin1/hugin/OptimizePanel.cpp
===
--- src/hugin1/hugin/OptimizePanel.cpp	(revision 5059)
+++ src/hugin1/hugin/OptimizePanel.cpp	(working copy)
@@ -82,7 +82,7 @@
 
 // local optimize definition. need to be in sync with the xrc file
 enum OptimizeMode { OPT_PAIRWISE=0, OPT_YRP, OPT_YRP_XYZ, OPT_YRP_V, OPT_YRP_XYZ_V,
-OPT_YRP_B, OPT_YRP_XYZ_B, OPT_YRP_BV, OPT_YRP_XYZ_BV, OPT_ALL, OPT_CUSTOM,
+OPT_YRP_B, OPT_YRP_XYZ_B, OPT_YRP_BV, OPT_YRP_XYZ_BV, OPT_ALL_NOTXYZ, OPT_ALL, OPT_CUSTOM,
 OPT_END_MARKER};
 
 OptimizePanel::OptimizePanel()
@@ -901,11 +901,26 @@
   SetCheckMark(m_d_list,false);
   SetCheckMark(m_e_list,false);
   break;
-  	case OPT_ALL:
+  	case OPT_ALL_NOTXYZ:
   // everything
   SetCheckMark(m_yaw_list,true);
   SetCheckMark(m_roll_list,true);
   SetCheckMark(m_pitch_list,true);
+  SetCheckMark(m_x_list,false);
+  SetCheckMark(m_y_list,false);
+  SetCheckMark(m_z_list,false);
+  SetCheckMark(m_v_list,true);
+  SetCheckMark(m_a_list,true);
+  SetCheckMark(m_b_list,true);
+  SetCheckMark(m_c_list,true);
+  SetCheckMark(m_d_list,true);
+  SetCheckMark(m_e_list,true);
+	  break;
+	case OPT_ALL:
+  // everything
+  SetCheckMark(m_yaw_list,true);
+  SetCheckMark(m_roll_list,true);
+  SetCheckMark(m_pitch_list,true);
   SetCheckMark(m_x_list,true);
   SetCheckMark(m_y_list,true);
   SetCheckMark(m_z_list,true);