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

Bug report: data paths were reset whenever RStudio was restarted #46

Open
XiaoyuZeng opened this issue Nov 19, 2020 · 3 comments
Open

Comments

@XiaoyuZeng
Copy link

Hi all.

I found that whenever I start RStudio, the data paths (e.g., raw_dir, data_dir) are reset to the default ones. And I have to run rave::rave_options() to correct those paths again. I'm running RAVE on win10, R4.0.2, RSudio 1.3.1056.

Can you replicate this bug in your environment?

Best regards,
Xiaoyu

@dipterix
Copy link
Collaborator

Thanks for reporting this issue @XiaoyuZeng I was trying to fix this issue yesterday. RAVE was developed on Mac and its Windows side was not well tested. Sorry for this obvious bug.

Could you try rave::check_dependencies() to update rave?

Also just in case you didn't noticed, there is a "Set Directory" link below the text input. You might want to click on it to save the settings.

image

Please let me know if the solution works or not.

Thank you!

@XiaoyuZeng
Copy link
Author

Thanks for reporting this issue @XiaoyuZeng I was trying to fix this issue yesterday. RAVE was developed on Mac and its Windows side was not well tested. Sorry for this obvious bug.

Could you try rave::check_dependencies() to update rave?

Also just in case you didn't noticed, there is a "Set Directory" link below the text input. You might want to click on it to save the settings.

image

Please let me know if the solution works or not.

Thank you!

Surely I click 'Set directory' whenever I reset the datapath using rave::rave_options().

I just tried rave::check_dependencies(), and it worked! The data path remained what I set before I restart RStudio.

Also, I found the 'Maximum number of CPU cores to be used' can be above 3 after the update (3 is the maximum before this update for windows 10).

Thanks for this update!

@dipterix
Copy link
Collaborator

@XiaoyuZeng great to hear that news!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants