Commit 477af33
feat: support visualization of graph queries by adding the --graph argument. (#94)
* Implement graph visualization with the --graph argument.
* Skip graph server tests when spanner-graph-notebook is missing. Also, add a test to bigquery magic for when spanner-graph-notebook is missing.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Fix lint errors and unit tests under nox.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add spanner_graphs to owlbot. Also, fix typo in package name in error message.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Fix owlbot entry
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add spanner_graphs as optional dependency with minimum version 1.1.1.
This is required for graph visualization.
* Change owlbot so spanner-graph-notebook is added to the config using python runtime 3.12 instead of 3.8, as spanner-graph-notebook does not support runtime version 3.8.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Fix typo
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Fix dependent package name: spanner_graphs -> spanner-graph-notebook
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add unit test coverage for the GraphServer object.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add more unit tests for convert_graph_data() to boost code coverage.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add more tests.
* Move get_ping() and post_ping() out of the GraphServer class, into the unit test
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Add unit test for handle_post_query() in GraphServer.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Fix lint, remove a line of dead code, mark test_post_query for skipping if spanner_graphs is not present.
* Remove more dead code in graph server, add test for --graph without spanner-graph-notebook present.
* Add unit tests for colab paths.
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Use pytest.raises() instead of try/except
* Add test coverage for the case where a graph query is run after the graph server is already running, due to another graph query having run previously.
* Add docstrings
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Move networkx and portpicker to extras under "spanner-graph-notebook".
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* Finish making graph dependencies optional
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
* fix port
* Convert graph server to singleton object.
* reformat
* Fix handling of null json elements.
* reformat
* Apply suggestions from code review
* Pin spanner-graph-notebook to exactly version 1.1.1, as subsequent changes to that repository broke our use of is.
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Tim Sweña (Swast) <swast@google.com>1 parent 60e1d57 commit 477af33
7 files changed
Lines changed: 1259 additions & 9 deletions
File tree
- packages/bigquery-magics
- bigquery_magics
- tests/unit
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| 56 | + | |
| 57 | + | |
56 | 58 | | |
57 | 59 | | |
58 | 60 | | |
| |||
61 | 63 | | |
62 | 64 | | |
63 | 65 | | |
64 | | - | |
65 | 66 | | |
66 | 67 | | |
67 | 68 | | |
| |||
97 | 98 | | |
98 | 99 | | |
99 | 100 | | |
| 101 | + | |
100 | 102 | | |
101 | 103 | | |
| 104 | + | |
102 | 105 | | |
103 | 106 | | |
104 | 107 | | |
105 | 108 | | |
106 | 109 | | |
107 | | - | |
108 | 110 | | |
109 | 111 | | |
110 | 112 | | |
| |||
114 | 116 | | |
115 | 117 | | |
116 | 118 | | |
| 119 | + | |
117 | 120 | | |
118 | 121 | | |
119 | 122 | | |
120 | 123 | | |
| 124 | + | |
121 | 125 | | |
122 | 126 | | |
123 | 127 | | |
| |||
391 | 395 | | |
392 | 396 | | |
393 | 397 | | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
| 402 | + | |
| 403 | + | |
394 | 404 | | |
395 | 405 | | |
396 | 406 | | |
| |||
425 | 435 | | |
426 | 436 | | |
427 | 437 | | |
428 | | - | |
| 438 | + | |
429 | 439 | | |
430 | 440 | | |
431 | 441 | | |
| |||
586 | 596 | | |
587 | 597 | | |
588 | 598 | | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
| 609 | + | |
| 610 | + | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
| 614 | + | |
| 615 | + | |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + | |
| 622 | + | |
| 623 | + | |
| 624 | + | |
| 625 | + | |
| 626 | + | |
| 627 | + | |
| 628 | + | |
| 629 | + | |
| 630 | + | |
| 631 | + | |
| 632 | + | |
| 633 | + | |
| 634 | + | |
| 635 | + | |
| 636 | + | |
| 637 | + | |
| 638 | + | |
| 639 | + | |
| 640 | + | |
| 641 | + | |
| 642 | + | |
| 643 | + | |
| 644 | + | |
| 645 | + | |
| 646 | + | |
| 647 | + | |
| 648 | + | |
| 649 | + | |
| 650 | + | |
| 651 | + | |
| 652 | + | |
| 653 | + | |
| 654 | + | |
| 655 | + | |
| 656 | + | |
| 657 | + | |
| 658 | + | |
| 659 | + | |
| 660 | + | |
| 661 | + | |
| 662 | + | |
| 663 | + | |
| 664 | + | |
589 | 665 | | |
590 | 666 | | |
591 | 667 | | |
| |||
634 | 710 | | |
635 | 711 | | |
636 | 712 | | |
637 | | - | |
| 713 | + | |
638 | 714 | | |
639 | 715 | | |
640 | 716 | | |
| |||
671 | 747 | | |
672 | 748 | | |
673 | 749 | | |
| 750 | + | |
| 751 | + | |
674 | 752 | | |
675 | 753 | | |
676 | 754 | | |
| |||
0 commit comments