-
Notifications
You must be signed in to change notification settings - Fork 25
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
cyto_transform(gs) Error: 'trans' must be a list of transformer objects #261
Comments
Look at RGLab/flowWorkspace#394 |
The fix is already in bioconductor latest release |
Thanks for commenting and providing a fix. Can you explain how to get the latest release of bioconductor? |
Did you try to update your current installation? |
Thanks, I followed your instructions but still getting the error:
|
Your flowWorkspace version is still 4.14.0. The update by Mike is available in 4.14.1. |
Thanks for explaining the issue @SamGG. Yes, my mac is arm64 (M2 chip). Is there an timeline as to when the version 4.14.1 of flowWorkspace will be available for MacOsS arm64? |
different architectures from bioconductor are inconsistent about scales package version
arm is lagging behind the rest of others, which is assume they will make patch pretty soon (feel free to send a message to their core team to report this hopefully get them act properly) |
Thanks for fixing this, @mikejiang. ARM builds typically lag behind other architectures by a week or so, so it should be available for download soon. @EmelineFavreau if you require a stable version of CytoExploreR to use in the meantime, I'd recommend using the docker image: https://github.com/djhammill/CytoExploreR-Docker. This docker image contains version 2.0.0 of CytoExploreR, so there might be some differences, please refer to the documentation and report any issues on the CytoExploreR GitHub page: https://github.com/DillonHammill/CytoExploreR. |
Hi all, can you please help me understand where the issue is? @DillonHammill, I tried to use your Docker container, but the Rstudio session aborted a couple of times when data were being produced (using the downstream analysis package flowAI). Additionally, the usually super handy gating 'cyto_gate_draw' failed to delivered. Indeed, the result of the mouse pointer was not precise (ie landed roughly at coordinates x-2 and y+3). @mikejiang , you refers to the scales package lagging to be updated for arm64 flavour. But on the website it looks like the version 1.3.0 is now available (https://cran.r-project.org/package=scales). Would you be able to update flowWorkspace now that scales is available, if this is possible at all? Many thanks! |
flowWorkspace source is already up to date, if you are asking for binary for mac from bioc, that has to wait until bioc team sort out the build problem in their mac server |
Frustratingly, arm64 is now two versions behind (still at 4.14.0) almost 3 months later. I cannot move forward on my project. Who exactly do I pester about this? |
From the build reports it looks like RProtoBufLib (a dependency of the flow tools) isn't getting installed due to a lock file error and that is cascading down to all the flow packages. This looks like a Bioc build system issue to me. Perhaps an email to the biconductor maintainers could help illuminate. |
That makes sense @gfinak, we have merged a fix to flowWorkspace that will work with new and old versions of scales (RGLab/flowWorkspace#397) but perhaps with the build errors that hasn't made its way to BioC yet. For users struggling with ARM64 at the moment, I'd recommend using one of the I haven't been able to test the image locally (I'm on AMD64) but the image has built successfully so it should provide a stable cytoverse working environment that includes CytoExploreR. @EmelineFavreau when using CytoExploreR through docker you need to make sure that your browser zoom is set to 100% (to avoid weird gating issues), you can also run Example docker command below, just need to modify the mount source path to the desired location on your machine:
|
Thanks @djhammill! I've never used Docker, so I decided to first try running with previous versions of scales (v. 1.2.0) and ggplot2 (v. 3.4.4) and that worked. So I can at least get the code to run again while waiting for the updates. |
Describe the bug
cyto_transform(gs)
is giving a new error:Error: 'trans' must be a list of transformer objects (generated by scales::trans_new method)
To Reproduce
Follow the vignette: https://dillonhammill.github.io/CytoExploreR/articles/CytoExploreR.html
Expected behavior
I expected the object to be created without error (which was the case two weeks ago when I last ran it)
sessionInfo():
Additional context
I have been running this code several times without an issue, and now I see this issue. I do not understand why. Is it something to do with the package scales?
The text was updated successfully, but these errors were encountered: