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

Update README and Documentation #362

Merged
merged 22 commits into from
Mar 11, 2025

Conversation

rob-shalloo
Copy link
Member

@rob-shalloo rob-shalloo commented Feb 19, 2025

This adds a link to the recently compiled lasy tutorial to both github and to the documentation on readthedocs. Addresses #353

Additionally, the README has been reshuffled to push the installation instructions further up the page. Also, specific instructions have been added to allow users to install the most up-to-date version of the code rather than the last official release

Finally, the laser_utils.py module has been added to the readthedocs.

@rob-shalloo rob-shalloo changed the title Add LASY tutorial to the README and the documentation [WIP] Add LASY tutorial to the README and the documentation Feb 19, 2025
@rob-shalloo rob-shalloo changed the title [WIP] Add LASY tutorial to the README and the documentation Update README and Documentation Feb 24, 2025
Copy link
Contributor

@MaxThevenet MaxThevenet left a comment

Choose a reason for hiding this comment

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

Thanks! See comments and suggestions below

Applied suggested changes

Co-authored-by: Maxence Thévenet <[email protected]>
Copy link
Contributor

@MaxThevenet MaxThevenet left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@MaxThevenet MaxThevenet merged commit af8f6b7 into LASY-org:development Mar 11, 2025
6 checks 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