<div dir="ltr">Yes, grouped feature classes seem like a pointless level of abstraction. You're right that it wouldn't allow selecting which features to use from within a group class, and would basically bundle everything together, which is not what we want.  </div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 9, 2016 at 8:09 AM, James Aylett <span dir="ltr"><<a href="mailto:james-xapian@tartarus.org" target="_blank">james-xapian@tartarus.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Aug 08, 2016 at 02:32:01PM -0400, Ayush Tomar wrote:<br>
<br>
> I am working on breaking down Features into sub-classes. Should each of the<br>
> features get their own sub-class, or should the grouping be done according<br>
> to type? i.e. query-document pair dependent, query-dependent and document<br>
> dependent sub-classes.<br>
><br>
> Using this approach makes more sense if we plan to add support for user to<br>
> include query-dependent and document-dependent features in future (we only<br>
> support query-doc dependent features at present), however these will remain<br>
> empty till the time we don't add features into them.<br>
<br>
</span>So I think the main driver for this is going to be: how in both cases<br>
would you choose which features to use? If everything is its own<br>
class, you just pass start/end iterators over objects. If there's a<br>
QueryDocumentPairFeatures class, how would you say "use *this* one but<br>
not *that* one"?<br>
<br>
It feels like grouped would be the worst of both worlds to me, but<br>
maybe I'm missing something.<br>
<br>
There's also not a huge point in building empty classes, so I'm not<br>
sure how grouped would be significantly different from what we have now.<br>
<span class="HOEnZb"><font color="#888888"><br>
J<br>
<br>
--<br>
  James Aylett, occasional trouble-maker<br>
  <a href="http://xapian.org" rel="noreferrer" target="_blank">xapian.org</a><br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-size:12.8px;font-family:arial"><font size="2" color="#666666">----------------------------------------------------------------------------</font></div><div style="font-size:12.8px;font-family:arial"><font size="2" color="#666666">Kind Regards,</font></div><div style="font-size:12.8px"><font size="2" color="#666666" style="font-family:arial">Ayush Tomar</font><font size="2" color="#0000ee" style="font-family:arial"> | <a href="http://ayshtmr.xyz" style="color:rgb(17,85,204)" target="_blank">My Webpage</a></font><font color="#666666" style="font-family:arial"> | <a href="https://in.linkedin.com/in/ayushtomar" style="color:rgb(17,85,204)" target="_blank">LinkedIn</a></font></div></div></div></div></div></div></div></div></div></div>
</div>