Adds multi-edge on multi undirected graphs.#273
Open
ceandrade wants to merge 194 commits intoWestHealth:masterWestHealth/pyvis:masterfrom
Open
Adds multi-edge on multi undirected graphs.#273ceandrade wants to merge 194 commits intoWestHealth:masterWestHealth/pyvis:masterfrom
ceandrade wants to merge 194 commits intoWestHealth:masterWestHealth/pyvis:masterfrom
Conversation
…-color Feature/add node font color
…operties-to-physics Feature/add setter properties to physics
Merge pull request WestHealth#5 from WestHealth/feature/add-setter-properties-t…
…ysics-config-window Feature/add toggle physics config window
Setting width='500px' actually creates iframe with height='500px' and vice versa.
Fix IFrame width and height mixed up
When another solver is chosen it wasn't reflected in the generated javascript. This resulted in barnesHut always being the active solver.
Set the solver when barnesHut isn't chosen
…-node-label-clickable Feature/make link in node label clickable
…hovering, live configuration
…eir way in to the JS
…is-fails-after-disable-physics-one-node Hotfix/i13 network vis fails after disable physics one node
Adjusted the typo in the arguments passed in the got_net variable. Made the call to read_csv download the CSV file.
…nd should allow some additional functionality going foreward.
Add filtering functionality to the network graphs
Experimental Merged into master: Thanks for all the great effort @js-konda @BerserkerGaruk
Update game of thrones example source
… defaulting self.height to 600px from 500px.
…ly situation where there needs to be a lib directory in the root of the local deployment is when pyvis is generating the html for a notebook and the notebook is serving the contents locally. This was some legacy code that was not as thouroughly checked when the last deployment went live and because of how it was coded the lib directory would get overwritten only when there was already a lib directory present.
… self.cdn_resources argument on how to actually generate the graph. Address WestHealth#199, WestHealth#175, WestHealth#189 by adding option to open web browser when the graphic is created. Updated warning message for notebook when writing the graphic.
… is handled when dealing with notebook generation.
Lib directory rewrite handling
…ional, serving as base for new code only.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As VisJS allows us to have multiple edges between the same pairs of nodes on multi undirected graphs, here we add such capability. Indeed, the changes in the code are trivial by commenting out / removing the code that checks and prevents the addition of multiple edges on undirected graphs. We also fixed/commented out the respective tests for such capability since it is disabled now.
However, no deep analysis was done to see the impact in other parts of the code.