A couple of (mini) bugs

Hi Mostapha,

A couple of things for your attention:

  1. On the LB Sunpath component when you add a conditional statement it writes “Conditiontional” on screen (I’m not sure if i reported this before, but just in case).

  2. I don’t understand why when i open the attached file it didn’t run the LB Analysis period. But if i close it and open again it does. I can also make it work if i change one of the sliders.

BTW i’ll llove to test the updated version … if you wish :slight_smile:

Thanks,

-A.

TMP.gh (45.2 KB)

Hi Abraham!

Sorry to get back to you late. I’m moving to a new city and it has been so busy these days. Thank you for reporting the bug! I fixed it. I can’t believe that I haven’t seen it this long!

I can’t repeat what you said on my system. It just works fine here… Does it give you any warnings or so?

Absolutely! I only didn’t want you to be upset as I haven’t yet applied all of your comments! Can you send me your email?

Best,

Mostapha

Hi Mostapha,

Good to hear from you.

Point 2: The only “warning” appears in the LB AnalysisPeriod component saying that i need first let the Ladybug fly … which is set to true from the beginning. I toggle to false and then to true again but nothing. Just moving any of the sliders connected to the LB_AnalysisPeriod does it.

My email: ayez@ar.technion.ac.il

Of course i won’t be upset. On the contrary. I know how to appreciate work in progress. I’m into that myself.

Thanks a lot,

-A.

Hi Abraham,

I see what’s happening. It is because of the order that Grasshopper executes the script. For some reason it does execute the LB AnalysisPeriod before LB_Ladybug component, so when LB AnalysisPeriod looks for Ladybug it is not there and it gives you the warning.

Just changing Ladybug to True afterward doesn’t help since LB AnalysisPeriod is not constantly listening to changes. You need to re-execute the LB AnalysisPeriod again after Ladybug is released to make it work.

I’ll send you an email. Thank you for your understanding and support!

Best,

Mostapha