-
Notifications
You must be signed in to change notification settings - Fork 364
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
New upgrade to 3.8 giving error in robyn_inputs #520
Comments
Thanks for reporting this bug @virithavanama I've updated main branch, can you please update and re-try? |
Thanks, @laresbernardo for the prompt fix :) |
Can you please share a reproducible example as I can't seem to be able to reproduce this one? |
Same issue with 3.8.0 for running robyn_run. If use the same data set with 3.7.2, no error. |
We can't reproduce the same error. Could you please share an anonymised dataset for us to debug. thanks |
@firatfrkn Thanks for providing the anonymous dataset for debugging. I've just pushed the fix. It runs now on my side. Can you please update the package and check. |
thanks a lot. It works for me now |
cool thanks for confirming. you can check the fix detail in the commit here |
@gufengzhou I still have the same problem after updating to v3.8.1. To update I ran the command - Robyn::robyn_update() |
strange. have you checked |
@gufengzhou Yes, I did. This time the error is - Error in {: task 1 failed - "non-numeric argument to binary operator" This error is different than the last time. I'm attaching the last time error here. |
Please provide your dataset anonymised and your model specification in robyn_inputs() and hyperparameters for debugging. you can send it to [email protected] |
Hey @AruunnAnanda , I managed to reproduce your error and just pushed a fix. Could you please update and retest. thanks |
Hey @gufengzhou, I unfortunately still have the same error. If I don't use calibration as an input the model RUNS FINE. If I start using calibration as an input the model throws an error |
ok that's an important information missing that you used calibration. I'll check and get back. |
@AruunnAnanda Please update and check |
@gufengzhou I still have the same issue :( I sent you the anonymised data via email to replicate and debug. |
thanks for the data. I've just pushed another fix and it runs on my macbook. please update and try |
there's been some fixes since then. can you update the package incl. dependencies, restart R and retry please? |
This should be fixed with the latest commits. Please update and check |
Please reopen if issue reoccurs. |
Hi Team,
I have upgraded to the new Robyn 3.8 version and getting the below error:
The text was updated successfully, but these errors were encountered: