Skip to content

Update project alumni data#160

Merged
dlebauer merged 6 commits intoPecanProject:masterfrom
NihalDR:contributers_v1
Mar 21, 2026
Merged

Update project alumni data#160
dlebauer merged 6 commits intoPecanProject:masterfrom
NihalDR:contributers_v1

Conversation

@NihalDR
Copy link
Contributor

@NihalDR NihalDR commented Mar 8, 2026

This PR updates the project alumni information by adding "Akash B V" information, Hamze Dokoohaki image aspect ratio update and adding image of Rohan Sasne

@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 8, 2026

I have even researched on finding image of tony Gardella Former Project Manager Graduate Student, University of Michigan.
But I could'nt find any it would be great if you can help me in getting the picture of him so that alumni page would be proper.

@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 8, 2026

This is how it looks after update
Screenshot 2026-03-09 023553
Screenshot 2026-03-09 023658
Screenshot 2026-03-09 023720
Screenshot 2026-03-09 023742

@dlebauer
Copy link
Member

dlebauer commented Mar 9, 2026

Thank you for contributing this. Because url-based photos have so frequently broken, I think it would make more sense to put the photos in the repository. That may be beyond the scope of this PR; if so, someone should create a new issue before merging this one.

@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 9, 2026

@dlebauer I can implement storing photos in repository as you said if you assign me to do I'll proceed or shall i open the issue to discuss on this?

@dlebauer
Copy link
Member

dlebauer commented Mar 14, 2026

@NihalDR Yes please, you can store images in the repository. Thanks!

@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 14, 2026

@dlebauer sure sir will work on it.

- Added images for Hamze Dokoohaki, Tanishq Jain, Rohan Sasne, and Akash B V
- Updated image URLs to use local images from /img/people/ instead of external URLs
- Added comments indicating which image URLs were updated
Copilot AI review requested due to automatic review settings March 16, 2026 18:07
@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 16, 2026

@dlebauer updated it as asked sir.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the alumni roster data used by the site’s Alumni page, switching select alumni photos to locally hosted assets and adding a new alumni entry.

Changes:

  • Replace external image URLs for Hamze Dokoohaki and Tanishq Jain with local /img/people/... assets.
  • Add a local image reference for Rohan Sasne.
  • Add a new Project Alumni entry for Akash B V (with local image reference).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

{
name: "Akash B V",
email: "akbv23cs@cmrit.ac.in",
role: "GSoC'25 Contributor & GSoC '26 Mentor",
role: "",
profession: "Staff Scientist, Indigo Ag",
img: "https://scholar.googleusercontent.com/citations?view_op=view_photo&user=91lPlxQAAAAJ&citpid=7",
img: "/img/people/Hamze Dokoohaki.png", // Updated to local image
Copy link
Member

Choose a reason for hiding this comment

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

@copilot could you please replace spaces with underscores in filenames, and set these new filenames to lowercase?

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: David LeBauer <dlebauer@gmail.com>
@dlebauer
Copy link
Member

Awesome, I went ahead and made all of the images local to avoid future breakage. Even found Tony's photo.

@dlebauer dlebauer merged commit ad91b51 into PecanProject:master Mar 21, 2026
3 checks passed
@NihalDR
Copy link
Contributor Author

NihalDR commented Mar 21, 2026

@dlebauer thankyou sir happy to hear that

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.

3 participants