Louis Maddox

Our test for this doesn't need to include the "safe" part to do with checking the message, pytest can handle the error raised (or not), so I'll just simplify it...

Well, 2 hours later and I don't think it's possible to reproduce a `BrokenPipeError` in pytest :expressionless: It always gets captured and captured as a `OSError: Bad file descriptor`. I...

Here is a test for regular print, not `rich.console.Console.print`. Maybe I need to mock something deeper in print to get it to work. When I swap the simple `Console` mock...

Request for review @willmcgugan Turned out it was basically impossible to capture this `BrokenPipeError` with pytest without resorting to `subprocess` :disappointed: Chalking that one up to a learning experience. I...

I think to finish this I need to add a test here https://github.com/pymc-devs/pytensor/blob/c10c376b173402eb06bed9aff56d0c77fd21dd79/tests/tensor/rewriting/test_math.py#L1945-L1971

One reason you may see that AttributeError is when you've only imported the namespace (as in, the directory named "pyflow") rather than imported the installed package. I resolved this with...

@jet1966 Hi Jeremy, If you're still interested in the idea please [email me](mailto:[email protected]) what "preview mode only" would look like - I'm not sure if that's to view only the...

Already on it! :+1: I'm writing a class `fsspec_reference_maker.csv.SingleCsvToPartitions`, following the `hdf.py` module's example. The class has the following method signatures: ```py def __init__( self, csv: BinaryIO, url: str, blocksize:...

I don't think it's right to initialise the `SingleCsvToPartitions` class with the `blocksize`: IMO it should be initialised with the list of offsets, and then 'edit' those on the `dask.Delayed`...

Hey Martin, that it has! New puppy arrived, sorry for the pause :sweat_smile: It's still up on my whiteboard, I'll bump it to the top of my weekend to-do list...