Generators in charts don't let me iterate more than once about mmpy HOT 5 CLOSED

The generator versus list issue in this issue is done for two reasons, though there are perhaps ways around both.

1. memory footprints - this is really not a major issue, given the current size of charts, but if they're very large...
2. response time - for current chart entities (artist) making a class instance doesn't call the api, but for others this may not be the case, suggesting a benefit to creating the object when used rather than when the chart is created.

Though now that I'm writing these reasons out, it occurs to me that this all may be adding up to a bit too much support for things that don't exist, since given the current calls I agree that a list comprehension makes more sense...

from mmpy.

I totally get where you're coming from, especially the ideas behind late binding for the purposes of reducing memory footprint and latency. In addition, I may only want the top 10 in a chart, and any other data is unnecessary (but note that I can't slice the generator anyway).

[So what we really need are Python closures, so that the Artist object instantiation can be late-bound instead of the list?]

Hum, I mostly brought it up to get at what you were thinking, and to advocate for dead-simple repeatability in a user-facing API.

from mmpy.

@atl :
[So what we really need are Python closures, so that the Artist object instantiation can be late-bound instead of the list?]

yes. I suppose the question then is which comprehension is a better approximation of what is desired, as a placeholder until such time as someone (probably me) can write something that looks a lot like closures...

from mmpy.

I was playing with some "how big is an Object" code in python, but calling vars() on a pre-API call Artist() shows how compact that object really is.

If you take a policy stand that other entities shall be as minimal as that (and hitting the API is a separate step), I'd place my vote for list comprehension.

from mmpy.

Alright, you've convinced me, until such time I (or some else) gets some proper closures in the charts class we'll change it over to a list comprehension, and to keep core entities small till you do something with them.

from mmpy.