> > The main issue issue is that #8414 changes the `lattice` method of
> > WeylGroups to `space` and Nicolas gave a reason why this might be
> > inaccurate sometimes. Perhaps that needs to be discussed here?  My
> > view is that `space` might be sometimes wrong but not egregiously,
> > whereas `lattice` is definitely bad terminology.  The only
> > alternative I can think of would be `module,` and if Nicolas
> > strongly prefers that, I will revise #8414 and the tutorial. But I
> > sort of like `space`.
>
> Let's see how the discussion on Sage devel goes. So far not much
> feedback ...

This might be mainly your call, since you reviewed #8414
with the comment about space being sometimes incorrect.

If you dislike "space" and prefer changing lattice -> module (or can
think of a better
alternative) I will revise the patch that way. But it seems to me that
"space" is better than
"lattice" here and usually correct.

But if #8414 is not to be merged before the documentation
patches I will have to revise #8442.

Dan

-- 
You received this message because you are subscribed to the Google Groups 
"sage-combinat-devel" group.
To post to this group, send email to sage-combinat-de...@googlegroups.com.
To unsubscribe from this group, send email to 
sage-combinat-devel+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/sage-combinat-devel?hl=en.

Reply via email to