The patch is in ParaView 3.98, and it was for ints as well. Do default domains for doubles show you the numeric entry in the GUI?<div><br></div><div>Cory</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Dec 6, 2012 at 3:58 AM, Biddiscombe, John A. <span dir="ltr"><<a href="mailto:biddisco@cscs.ch" target="_blank">biddisco@cscs.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-GB" link="blue" vlink="purple">
<div>
<p>I noticed that in pv 3.98 if you have an IntVector Property<u></u><u></u></p>
<p><u></u> <u></u></p>
<p><span style="font-family:"Courier New""> <IntVectorProperty<u></u><u></u></span></p>
<p><span style="font-family:"Courier New""> name="MaskPoints"<u></u><u></u></span></p>
<p><span style="font-family:"Courier New""> command="SetMaskPoints"<u></u><u></u></span></p>
<p><span style="font-family:"Courier New""> number_of_elements="1"<u></u><u></u></span></p>
<p><span style="font-family:"Courier New""> default_values="100" ><u></u><u></u></span></p>
<p><span style="font-family:"Courier New""> </IntVectorProperty><u></u><u></u></span></p>
<p><u></u> <u></u></p>
<p>Nothing appears in the GUI for the numeric entry unless you add
<u></u><u></u></p>
<p><u></u> <u></u></p>
<p><span style="font-family:"Courier New""> <IntRangeDomain name="range"/><u></u><u></u></span></p>
<p><u></u> <u></u></p>
<p>Which seems like the default domains were not picked up. Did something get lost, or was the patch only for double?<u></u><u></u></p>
<p><u></u> <u></u></p>
<p>JB<u></u><u></u></p><div><div class="h5">
<p><span lang="EN-US">-----Original Message-----<br>
From: Cory Quammen [mailto:<a href="mailto:cquammen@cs.unc.edu" target="_blank">cquammen@cs.unc.edu</a>] <br>
Sent: 12 October 2012 15:16<br>
To: Kyle Lutz<br>
Cc: Biddiscombe, John A.; ParaView Developers<br>
Subject: Re: [Paraview-developers] {Int, Double}VectorProperties now require a {Int, Double}RangeDomain element?</span></p>
<p><u></u> <u></u></p>
<p>Kyle,<u></u><u></u></p>
<p><u></u> <u></u></p>
<p>I forgot that merging is done through the gatekeeper review process.<u></u><u></u></p>
<p>Please ignore my request.<u></u><u></u></p>
<p><u></u> <u></u></p>
<p>Cory<u></u><u></u></p>
<p><u></u> <u></u></p>
<p>On Thu, Oct 11, 2012 at 11:36 PM, Cory Quammen <<a href="mailto:cquammen@cs.unc.edu" target="_blank"><span style="color:windowtext;text-decoration:none">cquammen@cs.unc.edu</span></a>> wrote:<u></u><u></u></p>
<p>> Kyle,<u></u><u></u></p>
<p>><u></u> <u></u></p>
<p>> Could you merge it? I don't have sufficient privileges.<u></u><u></u></p>
<p>><u></u> <u></u></p>
<p>> Thanks,<u></u><u></u></p>
<p>> Cory<u></u><u></u></p>
<p>><u></u> <u></u></p>
<p>> On Thu, Oct 11, 2012 at 5:26 PM, Kyle Lutz <<a href="mailto:kyle.lutz@kitware.com" target="_blank"><span style="color:windowtext;text-decoration:none">kyle.lutz@kitware.com</span></a>> wrote:<u></u><u></u></p>
<p>>> Looks good to me. I've +2'd it on gerrit.<u></u><u></u></p>
<p>>><u></u> <u></u></p>
<p>>> -kyle<u></u><u></u></p>
<p>>><u></u> <u></u></p>
<p>>> On Wed, Oct 10, 2012 at 9:39 PM, Cory Quammen <<a href="mailto:cquammen@cs.unc.edu" target="_blank"><span style="color:windowtext;text-decoration:none">cquammen@cs.unc.edu</span></a>> wrote:<u></u><u></u></p>
<p>>>> Kyle,<u></u><u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>> I've proposed a patch on gerrit<u></u><u></u></p>
<p>>>> (<a href="http://review.source.kitware.com/#/c/7866/" target="_blank"><span style="color:windowtext;text-decoration:none">http://review.source.kitware.com/#/c/7866/</span></a>) that essentially
<u></u><u></u></p>
<p>>>> implements John's suggestion and have added you as a reviewer.<u></u><u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>> Thanks,<u></u><u></u></p>
<p>>>> Cory<u></u><u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>> On Tue, Oct 9, 2012 at 10:21 AM, Biddiscombe, John A. <<a href="mailto:biddisco@cscs.ch" target="_blank"><span style="color:windowtext;text-decoration:none">biddisco@cscs.ch</span></a>> wrote:<u></u><u></u></p>
<p>>>>>> That is true. The new property widgets for each property are
<u></u><u></u></p>
<p>>>>>> created according to the property's domain. We can add in some
<u></u><u></u></p>
<p>>>>>> code to handle the case in which no domain is specified and just
<u></u><u></u></p>
<p>>>>>> create a simple text box. Would that work for your use-case? If
<u></u><u></u></p>
<p>>>>>> so, could you please submit a bug report and assign it to me?<u></u><u></u></p>
<p>>>>><u></u> <u></u></p>
<p>>>>><u></u> <u></u></p>
<p>>>>> Surely it would be much more intuitive to simply create a range domain for all int/double vector properties if one wasn't specified, you can always define<u></u><u></u></p>
<p>>>>> <IntRangeDomain name="range" /> And not specify a min max.
<u></u><u></u></p>
<p>>>>> Why not just use this as the 'default' unless the user actually overrides them. No real need to put it in the xml?<u></u><u></u></p>
<p>>>>><u></u> <u></u></p>
<p>>>>> I am aware that ints might be<u></u><u></u></p>
<p>>>>> <BooleanDomain name="bool"/> But that's an override. Yes?<u></u><u></u></p>
<p>>>>><u></u> <u></u></p>
<p>>>>> JB<u></u><u></u></p>
<p>>>>><u></u> <u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>><u></u> <u></u></p>
<p>>>> --<u></u><u></u></p>
<p>>>> Cory Quammen<u></u><u></u></p>
<p>>>> Research Associate<u></u><u></u></p>
<p>>>> Department of Computer Science<u></u><u></u></p>
<p>>>> The University of North Carolina at Chapel Hill<u></u><u></u></p>
<p>><u></u> <u></u></p>
<p>><u></u> <u></u></p>
<p>><u></u> <u></u></p>
<p>> --<u></u><u></u></p>
<p>> Cory Quammen<u></u><u></u></p>
<p>> Research Associate<u></u><u></u></p>
<p>> Department of Computer Science<u></u><u></u></p>
<p>> The University of North Carolina at Chapel Hill<u></u><u></u></p>
<p><u></u> <u></u></p>
<p><u></u> <u></u></p>
<p><u></u> <u></u></p>
<p>--<u></u><u></u></p>
<p>Cory Quammen<u></u><u></u></p>
<p>Research Associate<u></u><u></u></p>
<p>Department of Computer Science<u></u><u></u></p>
<p>The University of North Carolina at Chapel Hill<u></u><u></u></p>
</div></div></div>
</div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Cory Quammen<div>Research Associate</div><div>Department of Computer Science</div><div>The University of North Carolina at Chapel Hill</div><br>
</div>