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

Created nx_tutorial_script.py file #162

Merged
merged 4 commits into from
Mar 31, 2023
Merged

Conversation

Sreeyusha
Copy link
Contributor

Completed task #5 and created a python script file. @MridulS please guide me if I have done any mistakes. Thank you

@Sreeyusha
Copy link
Contributor Author

@MridulS I have created a python script file and commented on the output in the code itself. I have also attached the picture of the graph network that was coming as output when this code is compiled.
plot figure

@Sreeyusha
Copy link
Contributor Author

Hi, @MridulS I have linked task #4 commits to this branch. Is it ok to do that or I should create a separate branch and raise a separate PR? Please guide me if I have done any mistakes. Thankyou.

@MridulS
Copy link
Member

MridulS commented Mar 9, 2023

I have linked task #4 commits to this branch. Is it ok to do that or I should create a separate branch and raise a separate PR? Please guide me if I have done any mistakes. Thankyou.

It's fine for now, but it's recommended to use clean branches while doing changes :)

@Sreeyusha
Copy link
Contributor Author

I have linked task #4 commits to this branch. Is it ok to do that or I should create a separate branch and raise a separate PR? Please guide me if I have done any mistakes. Thankyou.

It's fine for now, but it's recommended to use clean branches while doing changes :)

Sure, I will keep this in my mind. Thank you @MridulS

@Sreeyusha
Copy link
Contributor Author

Hi @MridulS. I changed my code a little bit so that the output will be understandable way. Please guide me if you need more requirements in this code. I will make sure that I try my best to fulfill all your requirements for code. Please correct me if I am wrong. Looking forward to your guidance. Thank you.

@MridulS
Copy link
Member

MridulS commented Mar 31, 2023

Thanks!

@MridulS MridulS merged commit b4329d7 into networkx:main Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants