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

Raster Vision Tutorials #13

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

Conversation

NoaMillsUSDA-ARS
Copy link
Contributor

Adds series of 8 Raster Vision tutorials. Adds pygcdl tutorial. Adds bio and headshot for Noa. Fixes problem in previous PR with updates to image processing tutorial markdown file.

NoaMillsUSDA-ARS and others added 30 commits December 15, 2023 17:26
@aedawid
Copy link
Contributor

aedawid commented Jul 30, 2024

@NoaMillsUSDA-ARS thank you for your contributions. Our team will review your pull request soon, and then we get back to you with feedback or any questions.

Copy link
Contributor

@aedawid aedawid left a comment

Choose a reason for hiding this comment

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

Please review all the included images and adjust their sizes, as the current number and total size make the repository excessively large. In general, images should be no wider than 800-1000px for full-page width and 300-500px for a two-column layout, sufficient for quality display on the platform. Whenever possible, use a lighter graphic format, such as JPG, and aim to keep each file under 100-200KB to optimize loading times and reduce the overall repository size.

For example, the chip220_stride215.png with its current size of 665KB, can be opened in GIMP and exported as JPG which reduces its size to 178KB, just by changing the file format. The NoaMills.jpg file is in JPG format but its current size is 2225px x 3267px while the bio profile only requires not more than 240px x 240px.

These changes will help maintain a lightweight, efficient GitHub Pages site.
Thank you!

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