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

Add safety checks of .f #7

Open
TimTeaFan opened this issue Apr 19, 2022 · 0 comments
Open

Add safety checks of .f #7

TimTeaFan opened this issue Apr 19, 2022 · 0 comments
Assignees
Labels
next major release will be implemented in next major release safety check

Comments

@TimTeaFan
Copy link
Owner

TimTeaFan commented Apr 19, 2022

It is possible that the function in .f uses the same variable as specified in idx which defaults to i. There should be a check in place, checking this , raising an error with the advise to set idx to a variable that is not used in .f.

map(1:4,  function(x) {
  i <- 5
  x + i
  }) %>% as_loop()

Need to solve #6 before solving this issue.

There also be other safety checks in place for all possible inputs like .sel .inp1 etc.

@TimTeaFan TimTeaFan self-assigned this Apr 19, 2022
@TimTeaFan TimTeaFan changed the title Add safety check when idx is used in .f Add safety checks of .f Apr 20, 2022
@TimTeaFan TimTeaFan added the next major release will be implemented in next major release label Sep 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next major release will be implemented in next major release safety check
Projects
None yet
Development

No branches or pull requests

1 participant