-1

I have a function called Produce_Output. It takes an X variable and a Y variable then in R carries out some calculations, SQL data retrievals, saves a plot to a file location etc. the function itself doesn't produce anything but triggers other actions.

I want to run this function through a data frame which has been setup for it. The dataframe has 8464 obs and 2 variables. I would like to use the function and pass each 2 variables to the function an observation at a time.

calling the function as follows 

for (Data_To_Process) {
    Produce_Output(TableA$Column1, TableA$Column2)
}

I get the following error

Error in $<-.data.frame(*tmp*, "OND", value = c(3379L, 3121L, 1699L, : replacement has 8464 rows, data has 3

I read a post on here about the data having NULLs, I've checked that and it doesn't. I also don't understand what it replacing what with. I just want it to process the first row, then the next... then the next... as I said the function has no output but triggers other procedures using the two values inserted. Any help would be appreciated.

duckmayr
  • 16,303
  • 3
  • 35
  • 53
Jon
  • 1
  • 1
  • You are passing the entire columns, not rows. It could be something like `for(i in 1:nrow(TableA)){Produce_Output(TableA$Column1[i], TableA$Column2[i])}`. – Rui Barradas Aug 20 '18 at 16:32
  • 4
    We have no way to figure out the problem with your code because you don't show us your data or your code. – divibisan Aug 20 '18 at 16:32
  • 1
    Where did TableA come from and how does it relate to Data_To_Process? – Elin Aug 20 '18 at 17:42
  • 1
    Please share sample of your data using `dput()` (not `str` or `head` or picture/screenshot) so others can help. See more here https://stackoverflow.com/questions/5963269/how-to-make-a-great-r-reproducible-example?rq=1 – Tung Aug 20 '18 at 18:45

1 Answers1

0

Rui Barradas - You are perfectly correct that was the issue. I was aware of the index method for the loop but had only access to instances where one column was being used so didn't understand how to reference other columns in the syntax. Thanks for your help

"You are passing the entire columns, not rows. It could be something like for(i in 1:nrow(TableA)){Produce_Output(TableA$Column1[i], TableA$Column2[i])}"

Jon
  • 1
  • 1