-
Notifications
You must be signed in to change notification settings - Fork 2
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
synapser refactor phase a #171
Conversation
There appears to be a |
@anngvu, should we report this on Sage-Bionetworks/synapser? |
Reported in Sage-Bionetworks/synapser#331. I think for now best solution is to specify |
|
Update 2:
|
9f5d032
to
9b87c11
Compare
tryCatch({ | ||
view <- .syn$tableQuery(glue::glue("SELECT id,type,benefactorId FROM {fileview_id}")) | ||
view <- synapser::synTableQuery(glue::glue("SELECT id,type,benefactorId FROM {fileview_id}")) %>% | ||
synapser::as.data.frame() %>% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually do not need synapser::
Merging this into |
Update:
synapser
(>= 1.0.0) and < 2.0, suggesting you use1.3.0
, see linked issue discussion regardingsynSetAnnotations
. However, by the time everything is refactored, I trust thatsynapser
will have added backsynSetAnnotations
.brew
issue upstream that will be resolved eventually, see discussion at setup-r@master 403 error: Failed to download resource "qpdf" r-lib/actions#280