Chart names are inconvenient #4

merged 1 commit into from Mar 30, 2012


None yet
2 participants

atl commented Mar 30, 2012

I did a quick dict of chart names -> UUIDs to allow for more convenient initialisation of the Chart() object. A spot check on the events suggested I shouldn't enshrine those in library code right now (but those who know better would be better judges of that).


gearmonkey commented Mar 30, 2012

This seems like a perfectly reasonable thing to put in, and we can merge the pull. It is worth noting though that at some point, eventually, one day, there may be a means to get names-> uuid mappings of all available charts from the api, in which case this convenience dict should get dynamically populated, but functionality of that sort is a long ways off..

@gearmonkey gearmonkey added a commit that referenced this pull request Mar 30, 2012

@gearmonkey gearmonkey Merge pull request #4 from atl/master
Chart names are inconvenient

@gearmonkey gearmonkey merged commit 1eabfae into musicmetric:master Mar 30, 2012

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