Resource function for bcolz

[mrocklin]

No description provided.

[mrocklin]

Merging soon if no comment.

[cpcloud]

does this need to be assigned to a variable?

[mrocklin]

Yes! Although really I should be using os.path.join

On Thu, Oct 9, 2014 at 1:41 PM, Phillip Cloud notifications@github.com
wrote:

In blaze/bcolz.py:

@@ -103,3 +107,18 @@ def into(a, b, *_kwargs):
def into(a, b, *_kwargs):
a.extend_chunks(chunks(b))
return a
+
+
+@resource.register('.+.bcolz/?')
+def resource_bcolz(rootdir, **kwargs):

• rootdir.rstrip('/') + '/'

does this need to be assigned to a variable?

—
Reply to this email directly or view it on GitHub
https://github.com/ContinuumIO/blaze/pull/721/files#r18672694.

[mrocklin]

Oh, actually, I don't need it at all anymore.

On Thu, Oct 9, 2014 at 1:42 PM, Matthew Rocklin mrocklin@continuum.io
wrote:

Yes! Although really I should be using os.path.join

On Thu, Oct 9, 2014 at 1:41 PM, Phillip Cloud notifications@github.com
wrote:

In blaze/bcolz.py:

@@ -103,3 +107,18 @@ def into(a, b, *_kwargs):
def into(a, b, *_kwargs):
a.extend_chunks(chunks(b))
return a
+
+
+@resource.register('.+.bcolz/?')
+def resource_bcolz(rootdir, **kwargs):

• rootdir.rstrip('/') + '/'

does this need to be assigned to a variable?

—
Reply to this email directly or view it on GitHub
https://github.com/ContinuumIO/blaze/pull/721/files#r18672694.

[mrocklin]

Fixed