permlist behavior for mindist functions should be better defined

[js850]

I think if permlist is None it assumes that all the atoms are permutable. this is unintuitive and should probably be changed.

[ruehle]

I think we discussed that before. I would argue the opposite way: None means not specified -> let mindist decide what do to. an empty list [] would be explicitly setting there are no permutations. I agree that None could be ambiguous, but i think [] is more intuitive for no permutatons?

[js850]

That's what we decided before, but I'm changing my mind. It just seems so counterintuitive to have permlist==None mean that all atoms are permutable. Especially when a completely homogeneous system is not really all that common.

On 20 Aug 2013, at 18:35, ruehle wrote:

I think we discussed that before. I would argue the opposite way: None means not specified -> let mindist decide what do to. an empty list [] would be explicitly setting there are no permutations. I agree that None could be ambiguous, but i think [] is more intuitive for no permutatons?

—
Reply to this email directly or view it on GitHub.