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

A better README file #1079

Merged
merged 24 commits into from
Feb 9, 2023
Merged

Conversation

Sai-Suraj-27
Copy link
Collaborator

Updated the README file

Description

After Observing the README files of Tensorflow, NumPy, SciPy, and Pytorch. I thought these changes would make our README file much better.

  1. Changed the commands in the Installation and Hacking parts of the README file to code snippets that can be directly copied.
  2. Added Contribution Guidelines section, Which gives a brief about how to start contributing to Heat, Also added a Guide at the end for GSoC-related queries.
  3. Inspired by Tensorflows README, added a Try your first program script in the Getting started section
  4. Added some resources related to Parallel computing, mpi4py, and Heat in the getting started section.

Issue/s resolved: #1051

Changes proposed:

Type of change

  • Documentation update

Memory requirements

NA

Performance

NA

Due Diligence

  • All split configurations tested
  • Multiple dtypes tested in relevant functions
  • Documentation updated (if needed)
  • Title of PR is suitable for corresponding CHANGELOG entry

Does this change modify the behaviour of other functions? If so, which?

No

@Sai-Suraj-27
Copy link
Collaborator Author

Hi, everyone. I found that a few changes can be made to our README file to make it more better and useful for first-time viewers. After seeing the readme files of Tensorflow, Numpy...etc I made some changes to make our README file look better and more useful, I created this PR with those changes, any suggestions are welcome. So, you can merge it if you find it good. Also, please tell me if you want me to make any more changes to it, Thank you.

Copy link
Contributor

@ClaudiaComito ClaudiaComito left a comment

Choose a reason for hiding this comment

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

Thank you so much @Sai-Suraj-27 ! We're going to merge and make changes later if needed

@ClaudiaComito ClaudiaComito marked this pull request as ready for review February 6, 2023 08:58
@ClaudiaComito
Copy link
Contributor

Something is wrong with the CI matrix. A topic for our bore-a-thon @mtar ?

Copy link
Contributor

@ClaudiaComito ClaudiaComito left a comment

Choose a reason for hiding this comment

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

While we fix the CI matrix, a small change request. Thanks @Sai-Suraj-27 !

README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@ClaudiaComito ClaudiaComito left a comment

Choose a reason for hiding this comment

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

Thank you so much @Sai-Suraj-27 !

@Sai-Suraj-27
Copy link
Collaborator Author

Thank you so much @Sai-Suraj-27 !

Will you merge it? 👀

@ClaudiaComito
Copy link
Contributor

Thank you so much @Sai-Suraj-27 !

Will you merge it? 👀

Yes I will, sorry codecov!

@ClaudiaComito ClaudiaComito merged commit fade48d into helmholtz-analytics:main Feb 9, 2023
@JuanPedroGHM JuanPedroGHM added the documentation Anything related to documentation label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Anything related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide contributors' guidelines in TL;DR format
3 participants