Non-numeric argument to binary operator

moodymudskipper / flow

View and Browse Code Using Flow Diagrams

https://moodymudskipper.github.io/flow/

Other

395 stars 26 forks source link

Non-numeric argument to binary operator #7

Closed MichaelChirico closed 4 years ago

MichaelChirico commented 4 years ago

funflow::view_flow(data.table::fread)

Error in cumsum(cfc_lgl) * cfc_lgl : non-numeric argument to binary operator

moodymudskipper commented 4 years ago

It comes from this :) :

if (input == "" || length(grep("\\n|\\r", 
            input))) {
        }
        else { ...

I need to fix my code to support empty { calls, should be easy.

moodymudskipper commented 4 years ago

This is solved by https://github.com/moodymudskipper/funflow/commit/542fb3ebc43b9590d576b90e6de39bf5ceb8a11c

Note that the display has changed a bit, I added an exit not after the if blocks to join the yes and no before moving forward. It makes the view less compact but it's closer to the code, and solves some bugs.

github-actions[bot] commented 2 years ago

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue and link to this old issue if necessary.