<div dir="ltr"><div><div><div>I think it is a good idea to let multiple kinds of sets interact, but at the time I implemented all this I wasn't sure exactly how to make it work, so I took a conservative approach -- make it an error now; then we can make it a non-error later any way we want, without constraint. I don't want to give the impression I'm against interoperation.<br> <br></div>Here's what I propose to do now:<br><br></div>- rename set? to generic-set?; this predicate recognizes the new all-inclusive generic set type<br></div><div>- rename set-immutable? to set?; this predicate recognizes the pre-existing immutable hash set type<br> </div><div>- leave set-mutable? and set-weak? alone; they recognize the other two hash set types (on the mutability axis)<br></div><div>- allow multiple-set operations to combine equal-based hash sets and lists, since both use equal? for equality<br> <br></div><div>I don't know how to make other user-defined types recognize when they share a notion of equality. I'm not 100% sure that rejecting these operations when sets don't share a notion of equality is actually the right thing to do; sometimes it's best to just let the user take responsibility for whether putting the elements of one set into another will have meaningful results. But that restriction is documented and predates my changes, so I'm leaving it in unless someone wants to propose a better way. That means rejecting all other kinds of sets beyond hash sets and lists for union, intersection, etc.<br> </div><div class="gmail_extra"><br clear="all"><div>Carl Eastlund</div> <br><div class="gmail_quote">On Thu, Aug 22, 2013 at 12:27 PM, Matthias Felleisen <span dir="ltr"><<a href="mailto:matthias@ccs.neu.edu" target="_blank">matthias@ccs.neu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div style="word-wrap:break-word"><div><br></div><div>I can see why an eq-set and an eqv-set can't be union-ed without additional information about the desired result type. </div><div><br></div><div>But if we had two different implementation of the eq-set API we could easily define unions as derived code; that's the point of OO. </div> <div><div class="h5"><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><br><div><div>On Aug 22, 2013, at 12:14 PM, Carl Eastlund wrote:</div><br><blockquote type="cite"><div dir="ltr"><div><div>Set-union never worked for even different hash set representations. Even before I touched the code, the union of an eq set and an eqv set, for instance, caused a runtime error.<br> <br></div>Generics do not have multiple dispatch. That's not a mechanism we have right now. And "fallbacks" are for when there's no method implemented for a given receiver value, they're not particularly related to a question of "are these things the same type".<br> <br></div>I chose to keep the semantics that union and operations like it only work with the same representation. Partly because that's how things already were, and partly to set the precedent that generics authors don't have to write two versions of every method. Perhaps this wasn't the best idiom, but it's what I wrote. Perhaps there's a better idiom for fallbacks that makes this work more cleanly. Anyway, right now, generic sets are designed so you can use any one representation, but they don't combine representations.<br> </div><div class="gmail_extra"><br clear="all"><div>Carl Eastlund</div> <br><br><div class="gmail_quote">On Thu, Aug 22, 2013 at 9:03 AM, J. Ian Johnson <span dir="ltr"><<a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> No, it doesn't seem to be using the fallback in this case.<br> <br> ianj@sampson:~/racket/racket/bin$ ./racket -il xrepl<br> Welcome to Racket v5.90.0.8.<br> -> (set-union '() (set))<br> ; in-list: contract violation<br> ; expected: list?<br> ; given: (set)<br> ; [,bt for context]<br> -><br> <div><br> -Ian<br> ----- Original Message -----<br> From: "Sam Tobin-Hochstadt" <<a href="mailto:samth@cs.indiana.edu" target="_blank">samth@cs.indiana.edu</a>><br> </div><div>To: "J. Ian Johnson" <<a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a>>, "Carl Eastlund" <<a href="mailto:cce@ccs.neu.edu" target="_blank">cce@ccs.neu.edu</a>><br> Cc: <a href="mailto:dev@racket-lang.org" target="_blank">dev@racket-lang.org</a>, "Matthew Flatt" <<a href="mailto:mflatt@cs.utah.edu" target="_blank">mflatt@cs.utah.edu</a>><br> </div><div><div>Sent: Thursday, August 22, 2013 8:51:30 AM GMT -05:00 US/Canada Eastern<br> Subject: Re: [racket-dev] Lists aren't sets, but have set-like operations<br> <br> Wait, `set-union` of two different set representations doesn't work?<br> <br> Sam<br> <br> On Thu, Aug 22, 2013 at 8:07 AM, J. Ian Johnson <<a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a>> wrote:<br> > You misunderstand. I used set-union, but single dispatch saw '() and used list-union for the '() (set) combination.<br> > -Ian<br> > ----- Original Message -----<br> > From: "Sam Tobin-Hochstadt" <<a href="mailto:samth@cs.indiana.edu" target="_blank">samth@cs.indiana.edu</a>><br> > To: "J. Ian Johnson" <<a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a>><br> > Cc: <a href="mailto:dev@racket-lang.org" target="_blank">dev@racket-lang.org</a>, "Matthew Flatt" <<a href="mailto:mflatt@cs.utah.edu" target="_blank">mflatt@cs.utah.edu</a>><br> > Sent: Thursday, August 22, 2013 8:02:50 AM GMT -05:00 US/Canada Eastern<br> > Subject: Re: [racket-dev] Lists aren't sets, but have set-like operations<br> ><br> ><br> ><br> > But 'list-union' is not a generic operation so it isn't surprising that this didn't work. To do this generically, you'd need to use 'set-union'.<br> ><br> > Sam<br> > On Aug 22, 2013 7:59 AM, "J. Ian Johnson" < <a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a> > wrote:<br> ><br> ><br> > The problem manifested itself when I got an exception that in-list can't be called on (set), which really confused me. (set? '()) answered true, so it tried to do (list-union '() (set)), which failed.<br> > Generic sets as they are don't work generically. Some action should be taken. Either set? means what it once did, or we do some awfully slow multiple dispatch for set operations. My bias shows.<br> > -Ian<br> > ----- Original Message -----<br> > From: "Matthew Flatt" < <a href="mailto:mflatt@cs.utah.edu" target="_blank">mflatt@cs.utah.edu</a> ><br> > To: "Carl Eastlund" < <a href="mailto:cce@ccs.neu.edu" target="_blank">cce@ccs.neu.edu</a> ><br> > Cc: "J. Ian Johnson" < <a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a> >, "dev" < <a href="mailto:dev@racket-lang.org" target="_blank">dev@racket-lang.org</a> ><br> > Sent: Thursday, August 22, 2013 7:22:25 AM GMT -05:00 US/Canada Eastern<br> > Subject: Re: [racket-dev] Lists aren't sets, but have set-like operations<br> ><br> > How much should we prioritize backward compatibility in this case?<br> ><br> > One possibility is to make `set?' mean `hash-set?', and add<br> > `generic-set?' in place of the current `set?'. That's uglier,<br> > obviously, but it would be better if we want to prioritize backward<br> > compatibility.<br> ><br> > At Wed, 21 Aug 2013 19:14:06 -0400, Carl Eastlund wrote:<br> >> Ah, yes. The set? predicate no longer distinguishes a representation.<br> >> There are several predicates for the original set type, now called "hash<br> >> sets": set-eq?, set-eqv?, set-equal?, set-mutable?, set-immtuable?, and<br> >> set-weak?. I didn't add the basic "hash-set?", but perhaps I should. It's<br> >> a weird name, since "hash-set" and "hash-set!" are already existing,<br> >> unrelated functions.<br> >><br> >> Carl Eastlund<br> >><br> >><br> >> On Wed, Aug 21, 2013 at 7:08 PM, J. Ian Johnson < <a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a> > wrote:<br> >><br> >> > Okay, I can abide. However, that doesn't really get at my frustration. I'm<br> >> > using the set constructor, that appears to now be an immutable-custom-set<br> >> > with make-immutable-hash as its make-table. So what I'm looking for is not<br> >> > set?, but set-immutable?, as it's a distinct (family of) struct types that<br> >> > won't clash with the primitive data that I'm otherwise using.<br> >> > -Ian<br> >> > ----- Original Message -----<br> >> > From: "Carl Eastlund" < <a href="mailto:cce@ccs.neu.edu" target="_blank">cce@ccs.neu.edu</a> ><br> >> > To: "J. Ian Johnson" < <a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a> ><br> >> > Cc: "dev" < <a href="mailto:dev@racket-lang.org" target="_blank">dev@racket-lang.org</a> ><br> >> > Sent: Wednesday, August 21, 2013 6:58:56 PM GMT -05:00 US/Canada Eastern<br> >> > Subject: Re: [racket-dev] Lists aren't sets, but have set-like operations<br> >> ><br> >> ><br> >> > Ian, sets are now a generic datatype, like dictionaries. Association lists<br> >> > are dictionaries, and lists are now sets. They're also streams and<br> >> > sequences. They're not just "set-like".<br> >> ><br> >> ><br> >> ><br> >> > Carl Eastlund<br> >> ><br> >> ><br> >> > On Wed, Aug 21, 2013 at 6:56 PM, J. Ian Johnson < <a href="mailto:ianj@ccs.neu.edu" target="_blank">ianj@ccs.neu.edu</a> ><br> >> > wrote:<br> >> ><br> >> ><br> >> > I just wasted about 2 hours tracking down a bug that ended up being due to<br> >> > (set? '()) now evaluating to #t. I have no problems with set-union,<br> >> > intersection, etc. being defined for lists, but to treat lists as sets<br> >> > always is perverse to me. The contracts for set operations should use<br> >> > set-like? for (or/c set? list?) and keep the two constructions separate.<br> >> ><br> >> > This conflation is almost as bad as treating empty list as false.<br> >> ><br> >> > -Ian<br> >> > _________________________<br> >> > Racket Developers list:<br> >> > <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br> >> ><br> >> ><br> >> ><br> >> ><br> >> _________________________<br> >> Racket Developers list:<br> >> <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br> > _________________________<br> > Racket Developers list:<br> > <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br> <br> </div></div></blockquote></div><br></div> _________________________<br> Racket Developers list:<br> <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/dev</a><br></blockquote></div><br></div></div></div></blockquote></div><br> </div></div>