Huite

Results 199 comments of Huite

Another thing I noticed (after getting another e-mail): whether a discharge or rate is adding or removing water from the model isn't always mentioned in the docstrings. For example, the...

I'm adding ruff in #114, using the NPY rules as well. It always runs NPY201, but doing so expliclitly: ``` ❯ ruff check . --select NPY201 All checks passed! ```...

Since you mention it's a better example, maybe you could post the pictures for the file nonetheless? (The p:\11209585-dtp\voor_Caroline\01_scripts\post\python_differences\225deg-Car004-cropped.nc I mean) Having the MWE is helpful, but those pictures probably...

Ah okay I think I understand: the rivers look really fat. If I blow up the dpi: ```python fig, ax = plt.subplots(figsize=(10, 7), dpi=600) uds_sel.mesh2d_s1.ugrid.plot(vmin=vmin, vmax=vmax, edgecolors="none") ``` The rivers...

So in essence: you either have too small polygons, or too thick lines. You want the lines to fill up the holes between the polygons, but not so much that...

I agree, but it's a somewhat workable solution for now. I tried switching to other backends, but the white lines are drawn consistently. It's a matplotlib issue, so you could...

From matplotlib: here's a good explanation https://github.com/matplotlib/matplotlib/issues/9574#issuecomment-339532262 > This is a fairly difficult to solve problem in general. See e.g. https://computergraphics.stackexchange.com/questions/1824/what-is-illustrators-vector-rasterization-process. > > It's not, strictly speaking, because the edges...

See also: https://github.com/jklymak/contourfIssues This mentions: > If you are producing PDF files, setting the edgecolor='face' option in contourf and setting the linewidths=0.2 yields a good result on a 200-dpi monitor....

We should also make a note in the plotting docs.

I think it needs a different option for one of the keywords: [xarray.concat](https://docs.xarray.dev/en/stable/generated/xarray.concat.html)