Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

So far the labels can be only dict. Thus, we should implement the .items() function for NodeStat and EdgeStat.

See #233.

So far, the hyperedges labels are only drawn for Hypergraph and not for SimplicialComplex. Could you please give me some feedback whether it would make sense to add this functionality for SimplicialComplex as well?

@maximelucas

So far, I did not introduce any check for the node_labels and hyperedge_labels because I think the raised errors (from python) are already explicative. Let me know if I should improve them.

I think it's good for now!

Great idea, thanks Martina!
Having it for SimplicialComplex too would make sense yes. But there's already an SC case where it works in the notebook is there not?
Just left a comment in the notebook too.

In the notebook I only show how to use node_labels and not hyperedge_labels because I don't know if make sense to have a label for the simplices

Okay got it, because of potentially too many labels. I'd say in small SCs it might be useful to see them. In big ones with larger orders not really, but then the user can turn the labels off.

In any case, we're plotting only the maximal simplices, right @iaciac ? And the option to show their labels might helpful to the user.

Yes, we are plotting only the maximal order.. okay I will implement it as well then and come back to u

Added! However, it is less intuitive than for hypergraph because now if you pass a dictionary you don't know a priori which one will be the keys kept by the drawing function.. More specifically, with hyperedge_labels=True it works perfectly fine and shows the id labels. If you pass hyperedge_labels=SC.edges.size.asdict() does not work because the keys are different from the keys of H_ used in the plot.

Nice! And great point. Maybe add a note about this in the docstring? And then good to go I'd say.
(In the future we might only store maximal simplices or at least have a way to get maximal simplices which will make this better)

I'm already approving, feel free to merge when good for you.

Merged! 🎉 Grazie @mcontisc