search

albandechasteigner / GeniusLociForDynamo

Genius Loci is a package of 350+ custom nodes for Dynamo in Revit.
MIT License
45 stars 10 forks source link

GeniusLoci.Utilities.List SortNatural: Unexpected behaviour encountered #35

Closed andreasmweber closed 2 months ago

andreasmweber commented 4 months ago

Hello Alban, Thank you first of all for your fantastic work and your contribution to Dynamo packages. I have encountered an unexpected behaviour when using the List.SortNatural-Node in a Dynamo graph as shown in the two screenshots.

2024-07-28_Line Pattern_01

The red box indicates the only difference in the nodes of the two versions. In the second screenshot, alphabetical sorting works fine, but the filter after rearranging the list is leading to a completely different and unwanted result.

2024-07-28_Line Pattern_02

Is this behaviour intended? I am using Revit 2023.1 and DYNAMO v.2.16.2 with GeniusLoci 2023.7.13

albandechasteigner commented 4 months ago

Hi @andreasmweber,

Thanks for your kinds words. I can see a flaw in the logic used. You use the FilterByBoolMask node on the unsorted list. Instead, connect it to the List Sort Natural node. Sort