Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

synSetAnnotations not accessible in MacOS/Windows but works fine on Linux #331

Closed
anngvu opened this issue Feb 27, 2024 · 3 comments
Closed

Comments

@anngvu
Copy link

anngvu commented Feb 27, 2024

Operating system

NOT an issue on Ubuntu Linux, but an issue on Mac OSX and Windows 10.

  • synapseclient==4.0.0
  • synapser version ‘2.0.0.200’ (obtained with install.packages("synapser", repos = "http://ran.synapse.org"))

Description of the problem

> synapser::synSetAnnotations("syn51811164", annotations=list(foo="bar", baz=1))
Error: 'synSetAnnotations' is not an exported object from 'namespace:synapser'

Expected behavior

Works fine on Linux. Submits and prints the annotations as a list.

Actual behavior

> synapser::synSetAnnotations("syn51811164", annotations=list(foo="bar", baz=1))
Error: 'synSetAnnotations' is not an exported object from 'namespace:synapser'

Output of sessionInfo()R version 4.2.1 (2022-06-23)

Platform: aarch64-apple-darwin20 (64-bit)
Running under: macOS 14.3.1

Matrix products: default
LAPACK: /Library/Frameworks/R.framework/Versions/4.2-arm64/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] synapser_2.0.0.200

loaded via a namespace (and not attached):
 [1] compiler_4.2.1   rjson_0.2.21     here_1.0.1       Matrix_1.6-5     rprojroot_2.0.4 
 [6] tools_4.2.1      Rcpp_1.0.12      reticulate_1.28  codetools_0.2-18 grid_4.2.1      
[11] jsonlite_1.8.8   png_0.1-8        lattice_0.22-5  
@thomasyu888
Copy link
Member

Ah, apologies for the inconvenience. That function was removed as it wasn't working properly within the R client: https://r-docs.synapse.org/news/index.html. See changelog.

We should definitely have some sort of deprecation warning...

@anngvu
Copy link
Author

anngvu commented Feb 28, 2024

😦 Oh that's a big bummer; have always considered this to be a pretty core, at least one of the top 3 in usage for us. Updating the function docs would be helpful, thanks.

@anngvu anngvu closed this as completed Feb 28, 2024
@thomasyu888
Copy link
Member

thomasyu888 commented Feb 28, 2024

@anngvu we do plan on adding it back in after we refactor some things in the client. It just currently doesn't work due to the way reticulate converts objects that inherit from Python dict

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants