Jody Klymak

Results 580 comments of Jody Klymak

So what is the plan here? Should we just merge this and see if it works, and then fix any issues?

Only question I had was the empty `__init__.py` file in `tests/`?

... also - what is the plan for changing the files to ruff? Is that a second PR? I guess I thought that would come _before_ this PR, not after.

Hmmm we still build a search index? Would the faster to suppress building the index?

I took a quick look, and didnt' see a way to deactivate. It'd be great it if could be turned off, and you could use an old index if needed,...

The parent- child relationship lies in constrained layout. Outside of that context, gridspecs are not dependent on their figures, and certainly should not be inserted in the draw tree. Perhaps...

So... I don't think this is in the spirit of how pyglider is meant to work. We are not in the business of enforcing the metadata that gets put into...

For the repo size problem we could provide some guidance for the depth flag so most contributors don't need to download the whole thing. Eg --depth=500 gets you the last...

I'd not be in favour of this change. This is just a convention and in my mind Depth are rows, and profiles are columns because if you look at it...

> I had mostly thought about it from a plotting perspective, where (afaik) time or profile is always on the x-axis, and depth is on the y-axis. But fair enough,...