ladybug-tools / ladybug-legacy

:beetle: Ladybug is an environmental plugin for Grasshopper.
http://ladybug.tools
Other
192 stars 82 forks source link

New tab / category structure - not all components migrated #485

Closed ayezioro closed 4 years ago

ayezioro commented 4 years ago

Hi, I've updated to the last version on the github (This one). The folder identified by GH to upload them is now LB-Legacy or HB-Legacy, as opposed to the previous Ladybug or Honeybee. This works fine for most of the components, but still there are some, developed in C++ that are not yet migrated corrrectly and still preserve the old folder name. They are: Ladybug: Ladybug_ImageViewer - @MingboPeng Ladybug_SortByLayers - @MingboPeng False Start Toggle and @AntonelloDN all ladybut_ENVIMET located at the WIP tab. Honeybee: Honeybee_IntersectMassII - @saeranv? Honeybee_Item Selector Honeybee_ReadAnnualResultsIII - @MingboPeng

Hopefully they will be updated soon so alll the package will be fully functional.

Thanks, -A.

chriswmackey commented 4 years ago

@ayezioro ,

We are aware of all of the C# components and most of them were written by @MingboPeng and @AntonelloDN . @MingboPeng is aware of the change and is working on a new version of IronBug.

I think because @AntonelloDN has a newer version of the envi-met components that he's released under dragonfly legacy, we'll just say that the older version isn't distributed anymore, except in the very old Food4Rhino releases.

AntonelloDN commented 4 years ago

Hi @chriswmackey and @ayezioro,

Yes, we can remove old envimet components from Ladybug. It is better to have envimet stuff only under Dragonfly Legacy.

mostaphaRoudsari commented 4 years ago

Yes, we can remove old envimet components from Ladybug. It is better to have envimet stuff only under Dragonfly Legacy.

@AntonelloDN, can you give me a list of the components? or remove them yourself and send a PR. I almost know which ones are the ones but I'm not confident.

Thank you @ayezioro for reporting the issue. This is a change to open up ladybug and honeybee tabs for the [+] libraries.

AntonelloDN commented 4 years ago

Hi @mostaphaRoudsari ,

components are:

mostaphaRoudsari commented 4 years ago

Thank @AntonelloDN! I removed those and also removed "Ladybug_ENVI-Met Results Reader". https://github.com/ladybug-tools/ladybug-legacy/commit/65b56d98e97980fc9e5f987a5f94391e7615861c

chriswmackey commented 4 years ago

I know that @MingboPeng mostly addressed this as part of this commit.

However, the False Start Toggle seems to be a part of the Ironbug.LBHB_Legacy.gha that everything got wrapped into and this component is still showing up in the Ladybug tab (as opposed to LB-Legacy): image

Can you fix this @MingboPeng ?

chriswmackey commented 4 years ago

After clearing up some confusion, I think we can close out this issue thanks to this commit: https://github.com/ladybug-tools/ladybug-legacy/commit/f2419397010ff34d1a1967629e512a795233d226