dash-cytoscape icon indicating copy to clipboard operation
dash-cytoscape copied to clipboard

Upgrade to react-cytoscapejs 2

Open akx opened this issue 3 years ago • 3 comments

About

I checked that the Webpack builds pass, hoping that CI here would do the rest.

Cc

Description of changes

See above.

Pre-Merge checklist

  • [x] The project was correctly built with npm run build:all.
  • [x] 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 :dancer:'d the pull request. You can be one of these people if you are a Dash Cytoscape core contributor.

Reference Issues

See plotly/react-cytoscapejs#99 for the discussion around react-cytoscapejs 2.

akx avatar Sep 06 '22 13:09 akx

Thanks @akx. Looks like there are some build issues on CI... if you're motivated to dig in you're more than welcome, I've updated the permissions so it should work fine from your fork now, but I'm also happy to let this one wait a bit until someone from Plotly gets a chance to poke around here.

alexcjohnson avatar Sep 07 '22 19:09 alexcjohnson

@alexcjohnson Well, it worked for exactly one build... 😁

I think #179 is related here; since pip install has been set to --quiet I can't verify which version of dash is exactly being used, but it's obviously not one with https://github.com/plotly/dash/pull/1995 in, since the one build failed with

  File "/home/circleci/project/venv/lib/python3.7/site-packages/dash/dash.py", line 22, in <module>
    from werkzeug.debug.tbtools import get_current_traceback
ImportError: cannot import name 'get_current_traceback' from 'werkzeug.debug.tbtools' (/home/circleci/project/venv/lib/python3.7/site-packages/werkzeug/debug/tbtools.py)

akx avatar Sep 08 '22 07:09 akx

Yeah weird. Feel free to remove --quiet - and thanks for the nudge on #179

alexcjohnson avatar Sep 08 '22 13:09 alexcjohnson

Done in #193 - thanks @akx!

alexcjohnson avatar Jan 26 '24 21:01 alexcjohnson