Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ewentualne dalsze poprawki, które wyszły przy pisanie raport_data_info #120

Open
skonieczkak opened this issue Jun 11, 2023 · 2 comments
Assignees

Comments

@skonieczkak
Copy link
Collaborator

skonieczkak commented Jun 11, 2023

DO WSZYSTKICH, KTÓRZY PISALI FUNKCJE DO RAPORTU 1 (na razie v0 na moje gałęzi, jutro do review :)

  • ŚWIETNA ROBOTA, serio!
  • skoro raport to niejako ulotka naszego(?) pakiety to chcąc go zrobić porządnie(ładnie) wyszło kilka mankamentów, które przy rozwijaniu naszego pakietu musimy przemyśleć i poprawić
  • [tłumaczę się tu trochę dlaczego kod od raportu wygląda w ten sposób na razie]
  1. Funkcje czasem obsługują matrix, df, vector, zależnie od autora -> przyjmijmy zatem, że do raportu podajemu df. (zdecydowanie obługiwanie tego nie powinno być robione w raporcie)

  2. Funkcje outliers nie zawiera argumentu variables, wyrzuca błąd gdy zbiór danych zawiera NA

my_iris <- iris 
my_iris[1,1] <- NA
outliers(my_iris)
  1. Z funkcją redundant_cols coś jest nie tak
yes_no <- c("yES", "n",'y',"No",'yes',"nO")
true_false <- c('f','t','TrUe','FaLsE')

df <- data.frame(
 'a' = gsub('\\.',',',as.character(rnorm(10))),
 'b' = c(gsub('\\.',',',as.character(rnorm(5))),rnorm(5)),
 'c' = as.character(c(TRUE,FALSE), replace=TRUE, size=10),
 'd' = sample(yes_no, 10, replace=TRUE),
 'e' = sample(true_false,10, replace=TRUE),
 'f' = c(sample(yes_no,5, replace=TRUE),sample(true_false,5, replace=TRUE)),
 'g' = as.character(sample(1:100,10))
)
redundant_cols(df)
@szuvarska
Copy link
Collaborator

  1. Tam będzie wyrzucać kolumny, bo są traktowane jako ID (unikalne wartości nienumeryczne bądź numeryczne całkowite). Jeżeli ma ich nie wyrzucać, to trzeba wymyślić inną definicję ID. Masz jakiś pomysł?

@skonieczkak
Copy link
Collaborator Author

Hm, nie mnie o tym decydować, ale skoro to tak działa, to w wersji wydanej w czwartek zostawiłbym to w tej formie

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants