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

Historia #26

Open
lgelmo opened this issue Jun 11, 2018 · 1 comment
Open

Historia #26

lgelmo opened this issue Jun 11, 2018 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@lgelmo
Copy link
Collaborator

lgelmo commented Jun 11, 2018

Przepraszam, to jeszcze raz ja w sprawie dodawania wpisów do historii. Musiałem zmienić pomysł na zapisywanie do historii i w związku z tym proszę aby w każdej funkcji zamienić linijkę add_to_history("nazwa_twojej_funkcji") na linijkę o treści :

.hugoEnv$history[length(.hugoEnv$history)+1]<-deparse(match.call())

Mam nadzieję, że już nie będzie problemu z błędami. Przetestowanie na hugo_memorize_plot() wyszlo pozytywnie, więc są szanse, że teraz zadziała.

@kromash
Copy link
Contributor

kromash commented Jun 17, 2018

W niektórych funkcjach (np. hugo_get_secret ale też innych) przy wywołaniu podanej linijki występuje następujący warning:

In .hugoEnv$history[length(.hugoEnv$history)] <- deparse(match.call()) :
  number of items to replace is not a multiple of replacement length

Problem występuje gdy długość wywoływanej funkcji jest większa niż 60 znaków.
Funkja deparse ma domyślny parametr width.cutoff ustawiony właśnie na 60, a po przekroczeniu tej liczby dzieli podanego stringa i tworzy wektor stringów z którym nie radzi sobie przypisanie.

Należy zmodyfikować podaną linijkę na następującą:

.hugoEnv$history[length(.hugoEnv$history)] <- paste(deparse(match.call()), collapse='')

Linijka jest długa i niepotrzebnie komplikuje każdy skrypt, więc proponuję stworzyć dodatkową funkcję add_to_history która przyjmowałaby jako argument match.call() i wykonywała na nim powyższą linie

@Irbis3 Irbis3 added the bug Something isn't working label Jun 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants