Skip to content
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.

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

Enhancement/graph analysis #172

Merged
merged 14 commits into from
Jul 2, 2024
Merged

Conversation

yashbhesaniya
Copy link
Member

@yashbhesaniya yashbhesaniya commented Jul 2, 2024

Description

Related Backlog Item

Issue: #


Context


Checklist:

  • I have documented my changes
  • I have tested the changes and they work as expected
  • I have assigned this PR to someone
  • I have run make format to format my code

Additional references

Added a new constant `KEYWORDS_API_PATH` to the `constant.ts` file to define the API path for retrieving graph keywords. This will be used in the graph analysis feature.

Ref: #123
Signed-off-by: Yash Bhesaniya <[email protected]>
Added a new API endpoint `/graph_keywords/{graph_job_id}` to the `router.py` file in the `graph_creator` module. This endpoint reads a graph job by ID and returns important nodes. It uses the `analyze_graph_structure` function from the `graph_analysis` module to analyze the graph structure and retrieve the keywords.

Ref: #123
Signed-off-by: Yash Bhesaniya <[email protected]>
This commit optimizes the graph analysis process by modifying the `analyze_graph_structure` function in the `graph_analysis.py` file. Instead of calculating and returning all centrality measures, it now only returns the top nodes based on degree centrality, betweenness centrality, eigenvector centrality, and closeness centrality. This reduces the computational overhead and improves the performance of the graph analysis feature.

Refactor the `analyze_graph_structure` function to calculate and return the top nodes for each centrality measure.

Signed-off-by: Yash Bhesaniya <[email protected]>
This commit adds search functionality to the graph visualization feature. It introduces a search bar where users can enter keywords to search for specific nodes in the graph. The search is triggered when the user presses the Enter key. Additionally, the commit fetches graph keywords from the backend API and displays them as clickable chips below the graph. Clicking on a chip will automatically populate the search bar with the corresponding keyword.

Refactor the `GraphVisualization` component to include search functionality and display graph keywords.

Note: This commit message is based on the provided code changes and recent user commits.
Signed-off-by: Yash Bhesaniya <[email protected]>
Copy link
Contributor

@borges-filipe borges-filipe left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@borges-filipe borges-filipe merged commit 464894b into develop Jul 2, 2024
1 check passed
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.

2 participants