njtierney/naniar

geom_missing_point renders points cropped by plot limits as 'Missing'

Open

#58 ouverte le 15 juin 2017

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)R (43 forks)batch import
bugenhancementhelp wanted

Métriques du dépôt

Stars
 (557 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

@karawoo's response to my ggplot2 issue on missing data warnings made me twig this behaviour could also be an issue for geom_missing_point. It turns out that converting data outside the plot limits to NA seems to happen before our overloaded setup_data...(wat) creating a potentially misleading plot:

library(narnia)
library(ggplot2)
df <- data.frame(x = c(1:5) , y = 1:5)

ggplot(df, aes(x = x, y = y)) + 
  geom_missing_point() +
  xlim(c(0,4)) +
  ylim(c(0,5))

Her proposed work around using coord_cartesian does avoid the incorrect point categorisation:

ggplot(df, aes(x = x, y = y)) +                
geom_missing_point() +                         
coord_cartesian(xlim = c(0, 4), ylim = c(0, 5))

Guide contributeur

Direction de recherche
Examinez l'ordre des opérations dans la méthode setup data de geom missing point. Le problème est que les données en dehors des limites du tracé sont converties en NA avant l'exécution de setup data, ce qui entraîne une classification incorrecte. Regardez comment ggplot2 gère les limites et quand la conversion NA se produit. Correction potentielle : utilisez coord cartesian ou modifiez setup data pour qu'il s'exécute après la conversion NA.
Stack technique
r
Domaine
data visualization
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rggplot2
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55