Skip to content

Update links in README to point to GitHub#266

Open
aripitek wants to merge 1 commit intoArcInstitute:mainfrom
aripitek:patch-1
Open

Update links in README to point to GitHub#266
aripitek wants to merge 1 commit intoArcInstitute:mainfrom
aripitek:patch-1

Conversation

@aripitek
Copy link
Copy Markdown

No description provided.

@aripitek aripitek requested a review from a team as a code owner April 28, 2026 11:38
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request modifies several links and git clone commands in the README.md file, primarily by prefixing them with a specific GitHub user profile. The review feedback correctly identifies that these changes result in broken external links, invalid repository URL structures, and malformed SSH syntax for cloning the repository. Suggestions have been provided to revert these URLs to their original, functional states.

Comment thread README.md
Comment on lines +3 to +5
> Train State transition models or pretrain State embedding models. See the State [paper](https://github.com/aripitek/www.biorxiv.org/content/10.1101/2025.06.26.661135v2).
>
> See the [Google Colab](https://colab.research.google.com/drive/1QKOtYP7bMpdgDJEipDxaJqOchv7oQ-_l) to train STATE for the [Virtual Cell Challenge](https://virtualcellchallenge.org/).
> See the [Google Colab](https://github.com/aripitek/colab.research.google.com/drive/1QKOtYP7bMpdgDJEipDxaJqOchv7oQ-_l) to train STATE for the [Virtual Cell Challenge](https://github.com/aripitek/virtualcellchallenge.org/).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The external links (bioRxiv, Google Colab, and Virtual Cell Challenge) have been incorrectly modified by prefixing them with a GitHub user profile. This will result in 404 errors as GitHub does not host or proxy these external sites. These links should remain pointed to their original domains.

Suggested change
> Train State transition models or pretrain State embedding models. See the State [paper](https://github.com/aripitek/www.biorxiv.org/content/10.1101/2025.06.26.661135v2).
>
> See the [Google Colab](https://colab.research.google.com/drive/1QKOtYP7bMpdgDJEipDxaJqOchv7oQ-_l) to train STATE for the [Virtual Cell Challenge](https://virtualcellchallenge.org/).
> See the [Google Colab](https://github.com/aripitek/colab.research.google.com/drive/1QKOtYP7bMpdgDJEipDxaJqOchv7oQ-_l) to train STATE for the [Virtual Cell Challenge](https://github.com/aripitek/virtualcellchallenge.org/).
> Train State transition models or pretrain State embedding models. See the State [paper](https://www.biorxiv.org/content/10.1101/2025.06.26.661135v2).
>
> See the [Google Colab](https://colab.research.google.com/drive/1QKOtYP7bMpdgDJEipDxaJqOchv7oQ-_l) to train STATE for the [Virtual Cell Challenge](https://virtualcellchallenge.org/).

Comment thread README.md
Comment on lines +9 to +10
- Model evaluation framework: [cell-eval](https://github.com/aripitek/ArcInstitute/cell-eval)
- Dataloaders and preprocessing: [cell-load](https://github.com/aripitek/ArcInstitute/cell-load)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The repository links for cell-eval and cell-load use an invalid URL structure (/aripitek/ArcInstitute/). If these are intended to point to forks, the correct URL would be https://github.com/aripitek/cell-eval. However, for the main documentation, it is standard to link to the official organization repositories.

Suggested change
- Model evaluation framework: [cell-eval](https://github.com/aripitek/ArcInstitute/cell-eval)
- Dataloaders and preprocessing: [cell-load](https://github.com/aripitek/ArcInstitute/cell-load)
- Model evaluation framework: [cell-eval](https://github.com/ArcInstitute/cell-eval)
- Dataloaders and preprocessing: [cell-load](https://github.com/ArcInstitute/cell-load)

Comment thread README.md

```bash
git clone git@github.com:ArcInstitute/state.git
git clone git@github.com/aripitek:ArcInstitute/state.git
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The git clone SSH URL is malformed. The addition of /aripitek: before the organization name is not valid SSH syntax for GitHub. The correct format is git@github.com:owner/repo.git.

Suggested change
git clone git@github.com/aripitek:ArcInstitute/state.git
git clone git@github.com:ArcInstitute/state.git

Comment thread README.md
The model weights and output are licensed under the [Arc Research Institute State Model Non-Commercial License](MODEL_LICENSE.md) and subject to the [Arc Research Institute State Model Acceptable Use Policy](MODEL_ACCEPTABLE_USE_POLICY.md).

Any publication that uses this source code or model parameters should cite the State [paper](https://arcinstitute.org/manuscripts/State).
Any publication that uses this source code or model parameters should cite the State [paper](https://github.com/aripitek/arcinstitute.org/manuscripts/State).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The citation link has been incorrectly modified. It should point to the original arcinstitute.org domain rather than being prefixed with a GitHub profile.

Suggested change
Any publication that uses this source code or model parameters should cite the State [paper](https://github.com/aripitek/arcinstitute.org/manuscripts/State).
Any publication that uses this source code or model parameters should cite the State [paper](https://arcinstitute.org/manuscripts/State).

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.

1 participant