Added a 'graphs' method to the Dataset object

[iherman]

This is to handle issue #495.

[gromgull]

As far as I can recall it was removed intentionally, as the same functionality is available from the .contexts() method - inherited from the super-class. Let me check ...

[gromgull]

Yep - removed and mentioned in the commit message here: 1ed4feb

I would vote we fix the docs instead.

[iherman]

Well... I respectfully disagree. The very reason for having the Dataset object is because, if one uses RDFLib in terms of the RDF 1.1 standard, then the notion of "context" is unknown, whereas the term dataset and graph is defined and used all over the place. In this sense, from a usability point of view, the function name graphs() is clear, whereas the term contexts() is not.

I do not remember the history of this, it has been a very long time ago when this was done. It may well be that, originally, I actually used the method name graphs() and somebody changed to contexts() without me paying attention. That was my mistake then (I have to admit that I did not really look at rdflib in the past few years, my work focus at W3C have radically changed lately).

I am not sure what the decision process is in this case, and I do not want to make a big fuss about this; not being very active any more I do not think it is my place to do so. I just want to express my strong disagreement...

[gromgull]

decision process is a big word for what happens in RDFLib :)

Like you, I am not really using rdflib much myself anymore, and you make a fair point. I have no problem with reintroducing the method as an alias for contexts, it should then probably just do return self.contexts() though, and not duplicate the code.