cytoscape / RCy3

New version of RCy3, redesigned and collaboratively maintained by Cytoscape developer community
MIT License
49 stars 20 forks source link

Vignette bugs via Brett #33

Closed AlexanderPico closed 6 years ago

AlexanderPico commented 6 years ago

On Jun 25, 2018, at 3:48 PM, Brett Settle brettjsettle@gmail.com wrote:

Network functions and visualization Step 5, run install.packages(‘plyr’)

Fixed. Thanks!

Also uses the String app. Install step shows #installApp(‘STRING’) in a comment, but running installApp(‘STRINGapp’) worked for me

Fixed for future 3.7 users. Thanks!

openSession() failed to locate the YeastPerturbation session file

Probably because you are running dev 3.7 version :) It looks for flle relative to application: ./sampleData/sessions/Yeast Perturbation.cys

Updating Existing Scripts Should this go earlier in the tutorial? Section 5.3-5.6 uses the network loaded from 5.1, and fails on the network from 5.2, this is unclear No. These are only relevant to folks with old RCy3 scripts. Rare event. The sections are isolated examples, not a workflow.

updateNetworkInNDEx(isPublic=TRUE) fails to update the visibility of the network in NDEx. I think Jing removed the ability to update visibility, although it works from CyNDEx2 I'm fairly certain. Did this work for you?

Worked for me when I wrote the vignette. Haven’t tried in past month or so.

Group nodes at the end of the last vignette are extremely buggy. Not sure if just on my machine.

Ok.

AlexanderPico commented 6 years ago

Fixed by https://github.com/cytoscape/RCy3/commit/00d8866a9cbba8cf9241ddce26b4f3f96c1f3dcd