I just looked at the code, and it confirms the behaviour you describe.  I 
too find it unexpected.   If what I'm seeing in the code (ImagesPanel.cpp, 
ImagesPanel::CPGenerate() function ) is correct, it seems a somewhat 
trivial fix.   

Here's what I'd like:
---
No images selected and there are no existing control points of the current 
type (normal, vertical lines), process all images for new control points.
No images selected and there are one or more control points of the current 
type -- pop a dialogue, ask the user if they want to replace all control 
points in all images (yes/no).  If yes, delete all existing control points 
of the current type (normal, vertical line)  and process all images.
One or more images selected -- process only the selected images for 
additional control points.

In every case, duplicate control points should be eliminated after 
processing.   Somewhat less trivial fix, probably worthy of a separate 
function.

I haven't ever used the "vertical lines" control point feature, so I don't 
know how that *really* should be handled.

On Monday, March 7, 2022 at 12:49:57 PM UTC-8 [email protected] wrote:

> On Mon, Mar 7, 2022 at 3:34 PM Bruno Postle <[email protected]> wrote:
>
>>
>> I can't check right now, and I've only ever used this button to create 
>> control points for all images, but I seem to remember that you can select 
>> more than one photo and it will generate points just for that set.
>>
>
> Correct.  I was asking about what it does when just one photo is 
> selected.  Selecting zero does what you would expect and selecting 2 or 
> more does what you would expect.
>
> I know I could select the new one and the first old one and generate for 
> those two, then select the new one and the second old one and generate ...
>
> Also it is worth checking to see if the equivalent button in the fast 
>> preview does something more like what you are expecting.
>>
>
> Thanks, I'll check that in the code, but if that did what I want (for the 
> other tab), that would be a very strange UI.
>
> Unless someone explains why it is the way it is, I still think it should 
> be changed to be less surprising and more convenient, and I will do so in 
> hugin++
>
>
>

-- 
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
--- 
You received this message because you are subscribed to the Google Groups 
"hugin and other free panoramic software" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/hugin-ptx/3eee8370-31ab-4c61-a604-961d6e1b54den%40googlegroups.com.

Reply via email to