Carl Suster

This was adapted from an example in the python docs and solves test failures seen on the CI. We need to write to the underlying buffer object of the stream,...

Overwriting attributes converts the new value to the class of the old value

1

When setting an attribute that already exists (using `V() #> Attaching package: 'igraph' #> The following objects are masked from 'package:stats': #> #> decompose, spectrum #> The following object is...

Failure to read default config file silently ignored

1

In beets we noticed (see beetbox/beets#3264) that when confuse can't read the default config file it silently ignores the problem: https://github.com/beetbox/confuse/blob/04fd862963b0c1adb2cb81e4d97169869e88f2a9/confuse.py#L935-L939 Since beets was assuming that the file existed and...

`no_na()` always returns `FALSE`

15

The following code ```rust use extendr_api::prelude::*; fn main() { test! { let vals1 = Integers::from_values([Rint(1)]); dbg!(vals1.no_na()); let vals2 = Integers::from_values([Rint::na()]); dbg!(vals2.no_na()); let vals3 = Integers::from_values([Rint(1), Rint(2)]); dbg!(vals3.no_na()); let vals4 =...

Within `R/latex.s`, `format.df(x, ...)` relies on detecting the type of its argument `x` to determine how to access its attributes. Here are the key lines from that method: https://github.com/harrelfe/Hmisc/blob/6d9bd1f5bc254b0f95cc6fdfbb0e1ca4f761a06f/R/latex.s#L140-L144 The...

show_news(): Support NEWS in Rd and md formats and under inst

1

Fixes #2499. Exactly mirrors the search path documented in utils::news() for NEWS files rather than using the regex approach proposed in the issue. The existing code could only handle plain...

`show_news()` only checks `{pkg}/NEWS`

1

The `show_news()` function is hard-coded to check only for `NEWS` in a package directory: https://github.com/r-lib/devtools/blob/7cc2bdf74cbae90233fa6c84f221631d1876598d/R/show-news.R#L10 However (from the docs of `utils::news()`): >[...] it is attempted to read its news in...

My initial attempt at pie charts needs some more polish before it gets added back in. Specifically 1. It should be inset-aware and probably based on `StatAutomap` to position the...

Previously there was a `coord_automap_zoom()` that used a hacky workaround to give a sensible default zoom. The idea is that the normal `ggplot2::coord_sf()` keeps track of the bounding box but...

Support zoomed-out inset

1

Sometimes an inset is just about contextualising a zoomed-in map (e.g. https://stackoverflow.com/questions/34255534/inset-map-using-ggplot2). Theoretically this should already work (though an example is needed), but the inset frame needs to behave a...