Skip to content

Added image and updated description for Bhavya Shah #7

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.

Sign up for GitHub

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

Open
BhavyaShah1234 wants to merge 1 commit into
base: gh-pages
Choose a base branch
from
Open

Added image and updated description for Bhavya Shah #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
72 changes: 28 additions & 44 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,65 +16,49 @@ The website is created using a template from [Allan Lab](https://www.allanlab.or

---

## Getting Started (Clone & Run Locally)
## Getting Started (Fork & Add)

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

The Table of Contents at line 12 references "Git Workflow (Branches & Pull Requests)" section but this section no longer exists in the document. The content appears to have been merged into the "Getting Started" section. The Table of Contents should be updated to reflect the current document structure.

Copilot uses AI. Check for mistakes.

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

The Table of Contents at line 11 references "Getting Started (Clone & Run Locally)" but the section title has been changed to "Getting Started (Fork & Add)" at line 19. The Table of Contents anchor link needs to be updated to match the new section title.

Copilot uses AI. Check for mistakes.

### Prerequisites

- **Git** — [Install Git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git)
- **Ruby** (>= 2.5) and **Bundler** — [Install Ruby](https://www.ruby-lang.org/en/documentation/installation/)
- **Jekyll** — installed automatically via Bundler (see below)

### 1. Clone the Repository
### 1. Fork the Repository
> **⚠️ IMPORTANT: Make sure you are logged in to your GitHub account.

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

The blockquote is missing a closing double quote. The line should end with a closing quote after "account." to properly close the blockquote text.

Suggested change
> **⚠️ IMPORTANT: Make sure you are logged in to your GitHub account.
> **⚠️ IMPORTANT: "Make sure you are logged in to your GitHub account."**

Copilot uses AI. Check for mistakes.
- Go to [https://github.com/Logos-Lab/logos-lab.github.io](https://github.com/Logos-Lab/logos-lab.github.io)
- Click on the `Fork` button.
- Confirm the fork.

### 2. Clone your forked repository
```bash
git clone https://github.com/Logos-Lab/logos-lab.github.io.git
git clone https://github.com/<Your GitHub username>/logos-lab.github.io.git
cd logos-lab.github.io
```

### 2. Install Dependencies
### 3. **Create a new branch** for your changes:

> **⚠️ IMPORTANT: Do NOT commit directly to `gh-pages`.** All changes must go through a pull request.
```bash
gem install bundler # if you don't have bundler already
bundle install
git checkout gh-pages
git pull origin gh-pages # make sure you're up to date
git checkout -b <your-branch> # e.g. git checkout -b bshah43
```

### 3. Run the Site Locally
### 4. **Make your changes** (edit files, add images, etc.).
See `Updating Your Profile (Masters & PhD Students)` section

### 5. **Commit and push** your branch:
```bash
bundle exec jekyll serve
git add .
git commit -m "<Brief description of your change>"
git push

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

The git push command is incomplete and will fail. When pushing a new branch for the first time, you need to specify the remote and set the upstream branch. The command should be "git push origin your-branch-name" or "git push -u origin your-branch-name".

Suggested change
git push
git push -u origin <your-branch> # e.g. git push -u origin bshah43

Copilot uses AI. Check for mistakes.
```

The site will be available at **http://localhost:4000**. Changes to most files will auto-reload; if you edit `_config.yml`, you need to restart the server.

---

## Git Workflow (Branches & Pull Requests)

> **⚠️ IMPORTANT: Do NOT commit directly to `gh-pages`.** All changes must go through a pull request.

1. **Create a new branch** for your changes:

```bash
git checkout gh-pages
git pull origin gh-pages # make sure you're up to date
git checkout -b <your-branch> # e.g. git checkout -b update-omkar-profile
```

2. **Make your changes** (edit files, add images, etc.).

3. **Commit and push** your branch:

```bash
git add .
git commit -m "Brief description of your change"
git push origin <your-branch>
```
### 6. Merge changes in the main branch of **YOUR FORK**
- Go to `https://github.com/<Your GitHub username>/logos-lab.github.io`.
- See the pull requests section.
- Approve your pull requests and merge with your main branch.
Comment on lines +52 to +55

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

Step 6 instructs users to create a pull request in their own fork and merge it there, which is not the typical GitHub fork workflow. Normally, contributors should push their branch to their fork and then create a pull request directly to the upstream repository (Logos-Lab/logos-lab.github.io). The current instructions add unnecessary complexity and an extra merge step that could cause confusion.

Copilot uses AI. Check for mistakes.

4. **Open a Pull Request** on GitHub:
- Go to the repository on GitHub: https://github.com/Logos-Lab/logos-lab.github.io
- Click **"Compare & pull request"** for your branch.
- Set the base branch to **`gh-pages`**.
- Add a short description and request a review.
### 5. **Open a Pull Request** on GitHub:

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

Step numbering is duplicated: both step 5 (line 45) and step 57 are labeled as "5". Step 57 should be labeled as "7" to follow the correct sequence after step 6.

Suggested change
### 5. **Open a Pull Request** on GitHub:
### 7. **Open a Pull Request** on GitHub:

Copilot uses AI. Check for mistakes.
- Go to the repository on GitHub: [https://github.com/<Your GitHub username>/logos-lab.github.io](https://github.com/<Your GitHub username>/logos-lab.github.io)
- Click **"Compare & pull request"** for your branch.
- Set the base branch to **`gh-pages`**.
- Add a short description and request a review.

5. **Once approved and merged**, your changes will be live on the website.

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

Step numbering is inconsistent: line 63 uses "5." for a numbered list item when it should be "8." or should be unnumbered (using a hyphen) to properly follow step 7 above.

Suggested change
5. **Once approved and merged**, your changes will be live on the website.
- **Once approved and merged**, your changes will be live on the website.

Copilot uses AI. Check for mistakes.

Expand Down
11 changes: 7 additions & 4 deletions _data/students.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,14 @@
number_educ: 0
education1:

- name: Bhavya Shah
photo: s6.jpg
- name: Bhavya Minesh Shah
website: https://search.asu.edu/profile/5016411
photo: bhavya.jpg
Comment on lines +75 to +77

Copilot AI Feb 18, 2026

Copy link

Choose a reason for hiding this comment

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

The PR description mentions "images/teampics/bhavya.jpg" (with 's'), but the actual directory is "images/teampic/" (without 's'). Additionally, it mentions "students.yaml" but the actual file is "students.yml". While these are documentation issues in the PR description rather than the code, they should be noted for accuracy.

Copilot uses AI. Check for mistakes.
info: Master student
number_educ: 0
education1:
number_educ: 2
education1: Master of Science in Robotics and Autonomous Systems (AI) from Arizona State University
education2: Bachelor of Technology in Computer Science and Engineering from Vellore Institute of Technology, Vellore
joined: Spring 2026

- name: Anudeep Gottapu
photo: s6.jpg
Expand Down
Binary file added images/teampic/bhavya.jpg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading