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.

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

Add dependency graph entry and test files. #3

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Add dependency graph entry and test files. #3

wants to merge 18 commits into from

Conversation

sahelib25
Copy link

No description provided.

sahelib25 and others added 12 commits May 31, 2023 16:43
@sahelib25
Add entry for graph under varia_collection.
efb2553
@sahelib25
Upadate data_axs.json and code_axs.json
4d8cbcd
@elimkwan
Add dfs func for target node.
8093f91
@sahelib25
Update graph with parents of parents and multiple parents of output n…
f76c610 
…ode.
@sahelib25
Fix the issue of accessing data_axs.json files based on given target …
65e207f 
…entry.
@sahelib25
Add visual features to the graph: colouring, grouping and clustering.
b020883
@sahelib25
Merge remote-tracking branch 'upstream/master'
6101f26
@sahelib25
Add an extra functionality for extracting parents from output_entry_p…
bac5ed4 
…arents.
@sahelib25
Add more visual features to dependency graph.
549f744
@sahelib25
Enhance user readability of code_axs.py for graph entry.
50cce22 
Add test files to test parent entries and output parent entries of a target entry.
@sahelib25
Merge remote-tracking branch 'upstream/master'
032b5c7
@sahelib25
Add README.md for graph
d7bdfd9
elimkwan
elimkwan suggested changes Oct 10, 2023
View reviewed changes
Copy link
Contributor

@elimkwan elimkwan left a comment

Choose a reason for hiding this comment

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

Just minor requests on generalising the readme and cleaning the code :)

core_collection/varia_collection/graph/README.md Outdated Show resolved Hide resolved
core_collection/varia_collection/graph/README.md Outdated Show resolved Hide resolved
core_collection/varia_collection/graph/README.md Outdated Show resolved Hide resolved
core_collection/varia_collection/graph/README.md Outdated Show resolved Hide resolved
core_collection/varia_collection/graph/code_axs.py Outdated Show resolved Hide resolved
core_collection/varia_collection/graph/code_axs.py Show resolved Hide resolved
core_collection/varia_collection/graph/README.md Show resolved Hide resolved
core_collection/varia_collection/graph/README.md Show resolved Hide resolved
@sahelib25
Enhance user readability for code_axs.py and README.md.
9a0b2ee 
Remove absolute path to the image file.
G4V
G4V reviewed Nov 15, 2024
View reviewed changes
Copy link

@G4V G4V left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@G4V G4V G4V left review comments

@elimkwan elimkwan elimkwan approved these changes

@ens-lg4 ens-lg4 Awaiting requested review from ens-lg4

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sahelib25 @elimkwan @G4V