Skip to content

Conversation

@ohaanika
Copy link
Contributor

@ohaanika ohaanika commented Aug 9, 2019

Demo Checklist

Creating the R equivalent of the demos in the main directory of the repo:

  • usage-advanced.py
  • [WIP] usage-elements.py
  • usage-events.py
  • usage-stylesheet.py

Creating the R equivalent of the demos working correctly in Python:

  • usage-multiple-instances.py
  • usage-pie-style.py
  • usage-visual-style.py
  • usage-grid-layout.py
  • usage-circle-layout.py

Creating the R equivalent of the demons that are not fully functional yet in Python or are a WIP:

  • usage-animated-bfs.py
  • usage-breadthfirst-layout.py
  • usage-compound-nodes.py
  • [WIP] usage-concentric-layout.py
  • usage-concentric-social-network.py
  • usage-cose-bilkent-layout.py
  • [WIP] usage-cose-layout.py
  • usage-dag-edges.py
  • usage-edge-types.py
  • usage-elements-extra.py
  • usage-grid-social-network.py
  • usage-initialisation.py
  • usage-labels.py
  • usage-linkout-example.py
  • [WIP] usage-phylogeny.py
  • usage-preset-animation.py
  • usage-remove-selected-elements.py
  • usage-reset-button.py

About

Description of changes

Pre-Merge checklist

  • The project was correctly built with npm run build:all.
  • If there was any conflict, it was solved correctly
  • All changes were documented in CHANGELOG.md.
  • All tests on CircleCI have passed.
  • All Percy visual changes have been approved.
  • Two people have 💃'd the pull request. You can be one of these people if you are a Dash Cytoscape core contributor.

Reference Issues

Closes #[issue number]

Other comments

@xhluca xhluca marked this pull request as draft May 20, 2020 16:17
@xhluca xhluca requested a review from rpkyle May 20, 2020 16:17
@rpkyle rpkyle force-pushed the create-dashr-demos branch from 7750203 to 5a636ff Compare May 20, 2021 19:31
@rpkyle rpkyle changed the base branch from master to update-r-components May 20, 2021 19:32
@rpkyle
Copy link
Contributor

rpkyle commented May 20, 2021

I've removed the example which was unfinished (for now), and have edited out the very soon-to-be deprecated references to core component packages; I'll merge this into the v0.3.0 branch.

@rpkyle rpkyle marked this pull request as ready for review May 20, 2021 20:23
@rpkyle rpkyle changed the title [WIP] Create demos for R Create demos for R May 20, 2021
@rpkyle rpkyle force-pushed the create-dashr-demos branch from 9d66dc6 to 6df310c Compare May 20, 2021 20:26
@rpkyle
Copy link
Contributor

rpkyle commented May 20, 2021

Thanks again @ohaanika 🙃

@@ -0,0 +1,248 @@
dummyNodes <- list(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do the contents of /demosR/phylogeny serve any purpose?

I can't find any reason for them to be there, they don't affect the examples as far as I can tell.

🔪

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch -- I removed that demo (for now) because it was unfinished, will clean up this directory also.

rpkyle and others added 2 commits May 21, 2021 13:03
Co-authored-by: HammadTheOne <30986043+HammadTheOne@users.noreply.github.com>
Co-authored-by: HammadTheOne <30986043+HammadTheOne@users.noreply.github.com>
Copy link

@HammadTheOne HammadTheOne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃

@rpkyle rpkyle merged commit c5c62ad into update-r-components May 25, 2021
@xhluca xhluca deleted the create-dashr-demos branch May 25, 2021 18:27
xhluca pushed a commit that referenced this pull request Jun 21, 2021
* chore: use build:backends

* chore: update R package to v0.3.0

* chore: update CHANGELOG.md

* Create demos for R  (#68)

Co-authored-by: Aanika Rahman <aanika.rahman@mail.mcgill.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants