John Omotani

Results 77 issues of John Omotani

I came across this while investigating a slow-down using Makie (https://github.com/MakieOrg/Makie.jl/issues/3132). There seems to be a type-instability in `ccolor()` when using `--check-bounds=no`. The issue (at least my original problem) was...

bug

Some change in xarray-2022.9.0 introduced a significant performance regression (see https://github.com/boutproject/xBOUT/issues/213#issuecomment-1283669463). Possibly something related to the default option used to combine coordinates/metadata when combining two DataArrays/Datasets. There also seem to...

bug

Some operations, such as poloidal-plane plots/animations, can be annoyingly slow. This can happen even when a plot on the x-theta logical grid is fast, in which case the slow-down probably...

enhancement

In some routines, for example `BoutDataset.interpolate_parallel()`, many variables have to be merged back together. If they have coordinates, then these are checked for consistency. If coordinates are removed from the...

Add methods `BoutDataset.interpolate_to_new_grid()`/`BoutDataArray.interpolate_to_new_grid()` to interpolate a Dataset or DataArray to a new grid. The new grid should be defined by a new grid file created using the same equilibrium as...

enhancement
help wanted

Following this comment https://github.com/boutproject/xBOUT/issues/213#issuecomment-1397686005, converting a few members of `Region` from scalar `DataArray` to `float` reduces the number of errors. WIP because there are still a few errors in the...

bugfix

Some thoughts from @mikekryjak that we should follow up on: I think the examples are great, but they assume that the user already has some experience in xarray. I would...

help wanted
documentation

Might be useful for provenance tracking and reproducibility to know what version of `xBOUT` was used to produce some file. Similar to https://github.com/boutproject/boutdata/pull/79.

enhancement

Some of the examples are a bit outdated, e.g. using `"t_array"` instead of `"t"` (https://github.com/boutproject/xBOUT/issues/211#issuecomment-910399112). Should tidy them up...

documentation

From @ZedThree on Slack: Is there a way to silence this warning: "UserWarning: Size of toroidal domain as calculated from nz*dz (76.8 is not the same as 2pi*(ZMAX - ZMIN)...