Tim Hoffmann

Results 739 comments of Tim Hoffmann

I believe this is in arrow-width units, i.e. the arrow is 37.7 times longer than wide, but t.b.c. i.e. try modifying `quiver(..., width=...)` and check that this scales `length`.

> is it correct for the case of `'x' or 'y'`? I'm unclear what "it" is. Anyway, https://github.com/matplotlib/matplotlib/pull/28863#discussion_r1806405134 cover 'x' and 'y' cases as well.

> Hi, are there any changes to be made? Yes, but they were due to currently brittle handling of missing code references. I've made the relevant changes, so that you...

Thanks for the comparison images. Positioning looks correct. I can't tell for the linewidths. Obviously, that depends on the upscaling. Probably an additional helpful comparison would be hidpi Qt backend...

There are two Tk tests on MacOS failing. I'm unclear whether this is related to the PR.

We can also be more defensive and only change for linux as proposed [above](https://github.com/matplotlib/matplotlib/pull/28588#issuecomment-2353851803) and then create a followup issue to investigate OSX. It would be good to have at...

I believe a new standalone function would be suitable. Repurposing `figaspect` does not work because we'd end up with weird naming. Suggestion: ``` def figsize(*, width=None, height=None, aspect=None): ... ```...

I see. Can we come up with a reasonable specification that can be passed via `figsize`? The problem I see with `aspect` is that it interacts with figsize `figure(figsize=(6, 6),...

For only setting figure width supporting `figsize=(None, 4)` would be enough and more clear than `figsize=4`. We can easily do that - put in a PR if you like. But...

Ok, let's allow figsize tuples with at most one of the two entries being `None`. @arkapudota do you want to work on this? This does not restrict our ability to...