Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate the Title and Legend Outputs for All Graphic Components #131

Open
arkdanielnielsen opened this issue Feb 10, 2015 · 13 comments
Open

Comments

@arkdanielnielsen
Copy link

  1. Something goes wrong when I want to re-color a result from "Color Zones by EP Result".
  2. I am not sure in which component things goes wrong. It seems to be something with the meshing? Or is it the "Re-Color Mesh"?
  3. As seen in the image it is 8 identical simple shoebox'es. I assume they should have the same mesh size, but that is not the case. Original the geometry derived from a Rulled Surface from two straight lines.
  4. Why I am using is just to remove the legend text or want is called Analysis Text in the re-color mesh component. Would it be possible to have the Analysis text as a out in the different original components like the "Color Zones by EP Result" etc? That would in many cases be quite useful - at least what I think ;)
    ladybug_re-color-mesh-01
@chriswmackey
Copy link
Member

@arkdanielnielsen ,

  1. Can't you just use the legendPar of the "Color Zones by EP Result" component? Are you saying that is not working?
  2. Just don't graft the list of meshes coming out of the colorZones component and then the lists should probably align. Alternatively, you can graft the analysis result.
  3. The auto-meshing algorithms of GH might divide something up into quads or triangles. It is a difficult process to control unless you make the mesh yourself from scratch.
  4. Ok. I understand the answer to my fist question. I do not understand what you mean by Analysis Text. You you mean have title and legend text separately?

@chriswmackey
Copy link
Member

Ah wait. I just understood why the reColorMesh does not work. Here's a workaround but you probably don't want to use the recolor mesh component in this case.
image

Would having separate outputs for title and legend text fix this issue for you? If so, that seems like the best route.
-Chris

@arkdanielnielsen
Copy link
Author

The thing is, I want to get rid of the Analysis Title (it was a mistake I wrote Analysis text) and keep the legend. To do that I have to cull away the text but not in all cases the length of the analysis title is the same, so that makes it difficult to use that method. This is not only a wish for "Color Zones by EP Result" but in general for all components with legends. I think it would be easier to just remove or place the Analysis Title without using the reColorMesh.
So the best route would diffidently be a output for Analysis Title!

@chriswmackey
Copy link
Member

@arkdanielnielsen ,
I agree with you that it would be nice to have the outputs separate and I just made the change for the color zones and color surfaces component.
image

If you draw up a list of components where this grouping of legend and title outputs happens, I will begin working my way down it (it looks like this issue will take a lot longer to close than initially anticipated :). Feel free to order the list in terms of the components that you would most like to see the change and I will get to those first.

-Chris

@arkdanielnielsen
Copy link
Author

Looks good, thanks.
It sound like a deal, that I make the list. Maybe I could add the lines if it is the same lines there is needed in the different components?

@arkdanielnielsen
Copy link
Author

@chriswmackey finally I had time. In the gh file I have marked all component where the output of AnalysisTitle could be useful.
https://www.dropbox.com/s/3kni0fqm0ozt7w0/all_components_LBHB.gh?dl=0
Is it only to add the below lines? If so I could try to add them in the components.

analysisTitle = []
for count, item in enumerate(legendInit):
    if count == 0:
        legend.append(item)
    if count == 1:
        for srf in item:
            legend.append(srf)
    if count == 2:
        for srf in item:
            analysisTitle.append(srf)

@mostaphaRoudsari
Copy link
Member

@chriswmackey @arkdanielnielsen what is up with this issue? should we close this?

@arkdanielnielsen
Copy link
Author

@mostaphaRoudsari It is about separating the Analysis Text from the legend output in two separated outputs. To make it easier to remove the text of legend if it is not wanted. And I think it could be useful for all the components with legends.

@chriswmackey chriswmackey changed the title Bug or just mee !? "Re-Color Mesh" Separate the Title and Legend Outputs for All Graphic Components Mar 2, 2015
@chriswmackey
Copy link
Member

@arkdanielnielsen and @mostaphaRoudsari ,

I have changed the name of the issue to be closer to what we have decided to do. In my view, it is a long-term issue as I have a lot of other priorities on my list and there are a bunch of components to change. All that has to happen is that the code that @arkdanielnielsen posted has to go into a lot of the components and a new output has to be added to them. @arkdanielnielsen , if you want to take this up on your own, that would be swell.

Let me know your anticipated time frame and if you run into any issues.
-Chris

@chriswmackey chriswmackey removed their assignment Mar 2, 2015
@chriswmackey
Copy link
Member

I just realized that I cannot assign the issue to @arkdanielnielsen . I will put myself back as asignee.

@chriswmackey chriswmackey self-assigned this Mar 2, 2015
chriswmackey pushed a commit that referenced this issue Jul 19, 2015
As requested by @arkdanielnielsen here:
ladybug-tools/honeybee-legacy#251

This commit also addresses the request for separate title/legend outputs
here:
#131
@chriswmackey
Copy link
Member

I have gotten this separation of title and legend working now for a large fraction of the Ladybug components. I feel that I should just finish this off for the next stable release.

@chriswmackey
Copy link
Member

This feature will be addressed with the new core Ladybug functions that @mostaphaRoudsari is putting together and so I will let this issue rest until this core development is finished.

@chriswmackey
Copy link
Member

This is a monumental task that we will fix in LB+ but I welcome anyone who wants to implement this on any of the legacy components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants