8000 Refactor network visualization to use pre-fetched statements by prasham21 · Pull Request #232 · gyorilab/indra_cogex · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Refactor network visualization to use pre-fetched statements #232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 19 commits into from
Jun 11, 2025
Merged

Refactor network visualization to use pre-fetched statements #232

merged 19 commits into from
Jun 11, 2025

Conversation

prasham21
Copy link
Contributor

This PR refactors how network visualization data is generated, with two key improvements:

1. Single Query for Tabular and Network Views

Previously, the application executed duplicate database queries - one for the tabular statement display and another for the network visualization. This PR implements a more efficient approach:

  • Added a store_hashes_in_session parameter to render_statements to store statement hashes in the session
  • Updated route handlers for GO, paper, and subnetwork explorers to enable hash storage
  • Modified get_network to use statements from the session instead of executing a new query

2. Enhanced Relationship Display

The previous implementation showed only one relationship between entity pairs, hiding the rich diversity of biological interactions. This PR enhances relationship display:

  • Changed edge deduplication to use (source, target, statement_type) triplets instead of just (source, target) pairs
  • Modified statement matching to consider relationship types
  • Ensured all statements from the tabular view are properly represented in the network

@prasham21 prasham21 force-pushed the refactor-network-graphs branch from 24a2af3 to 0956e6b Compare May 15, 2025 13:04
Prasham Marfatia added 19 commits June 11, 202 8000 5 15:50
@bgyori
check if target_id is not None before trying to split it
161ae0e
@bgyori
Removing unncessary parameter examples fron the dictionary
5a9c64e
@bgyori
Refactor to use sessions based apporach for get_network_for_statement…
ec49c07 
…s and create and endpoint function for network visualization
@bgyori
Change button name
9134351
@bgyori
Add statement prefix
d1429c8
@bgyori
Modify payload properties
66af3c8
@bgyori
Refomrat file
da393a9
@bgyori
Remove unnecessary parameters
4711df7
@bgyori
Modify template to use prefix specific payload
aba9c01
@bgyori
Deduplicate Complex statement edges by treating them as undirected pairs
4fbd11f
@bgyori bgyori force-pushed the refactor-network-graphs branch from 49fcfbc to 4fbd11f Compare June 11, 2025 19:51
@bgyori bgyori merged commit 3ec9c58 into gyorilab:main Jun 11, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@prasham21 @bgyori
0