On Sat, 13 Jun 2015, Johan S. R. Nielsen wrote:

I think that the default of returning a sublattice makes most sense by far.

Yes, if we have both then this should be the default.

In case the speed difference is very significant (why would this be the
case?),

At least for now we don't have parameter like trust_me_i_know_what_i_am_doing=True for posets or lattices. This can be seen for example with C500=Posets.ChainPoset(500), and after that even C500.join(100,200) will take few seconds for first time. That means that ChainPoset() makes a digraph, that is tested to be a poset, then tested to be a lattice by creating join- an meet-matrix and then those matrices are thrown away.

(Funnier example is C500.is_lattice() taking few seconds, even if it's class is lattice.)

one could add an optional argument to the function to just
return a list. Could be named "as_list = False", I guess.

Yes, if the choises are - and will be - a list and a lattice. But maybe somebody wants to return a poset or a set?

 * * *

Anyways, I'll do these to return a lattice at least for now.

--
Jori Mäntysalo

Reply via email to