graphlab-fr/cosma

[Enhancement] Display link type on links.

Opened this issue · 3 comments

zyansheep commented

Which part of Cosma is your request about?

  • command-line interface
  • cosmoscope (HTML file)
  • documentation
  • other

Describe your request:

I'm not sure if this isn't already possible, but I would like to be able to visualize the names of links between nodes in the graph view.

infologie commented

@zyansheep Thank you for suggesting this. Indeed, Cosma does not display link types in the graph view. I will discuss this with @Myllaume.

zyansheep commented

I want to try to implement this myself, is there documentation I can look at for the code? Or do I just have to jump in and try to read it myself? If its the second, would it be too much to ask for an overview of which code (files) might have to be changed / added to implement this feature?

infologie commented

@zyansheep There are a couple of hurdles:

  • We have not yet created developer documentation. I'm sorry about this because a few people are starting to want to contribute. The code contains a lot of comments, and follows a classic model-view-controller structure, so some people may find their way through. But I'm no programmer. So if someone needs guidance, @Myllaume is the only one who could answer and he's currently on leave (for a very happy but time-consuming reason).

  • I would like this to be an optional enhancement. Users should be able to choose to generate link labels or not. When they choose to generate them, they should also be able to toggle link label visibility on/off in the graph. This means modifying more than one file, and make UX decisions. @Myllaume will want to review the code for homogeneity, and will want to tweak the UX. So we won't be able to make it a quick fix.

So you're welcome to try! But it might take a while to move from idea to pull request, and then to merging.

edit: forgot to mention that a few colleagues of mine have suggested this enhancement too. So it's on the list of things to explore. I don't know about the feasibility, so if you tackle this, let us know!