Skip to content

U/hkashgar/ug negishi new - #266

Merged
Guangzhen-Jin merged 27 commits into
mainfrom
u/hkashgar/ug_negishi_new
Jul 14, 2026
Merged

U/hkashgar/ug negishi new#266
Guangzhen-Jin merged 27 commits into
mainfrom
u/hkashgar/ug_negishi_new

Conversation

@haniyeka

@haniyeka haniyeka commented Jul 9, 2026

Copy link
Copy Markdown
Member

Finally Negishi documentation migration is done. Please, let me know if you see an issue and I will fix them.

@haniyeka
haniyeka requested a review from Guangzhen-Jin July 9, 2026 17:10

@Guangzhen-Jin Guangzhen-Jin Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some of the content are still using "Bell" information. e.g. "Bell-A nodes", number of nodes, and GPU types:

"made up of 446 Bell-A nodes"

Should be: "450 Negishi-A nodes".

"6 Bell-B nodes which have four times as much memory as a standard Bell-A node"

Should be: "6 Negishi-B nodes … standard Negishi-A node"

"two Zen 2 AMD EPYC 7763 64-core processors"

Negishi uses AMD EPYC 7763 "Milan" = Zen 3, not Zen 2 (Zen 2 = Rome = EPYC 7xx2 series). Should be "two Zen 3 AMD EPYC 7763 64-core processors".

"Each of these nodes contains two AMD MI210s"

Should be three AMD MI210 GPUs (64GB) per Sub-cluster G node.

"two Zen 2 AMD EPYC 7313 16-core processors"

Should be "two Zen 3 AMD EPYC 7313 16-core processors".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

@Guangzhen-Jin Guangzhen-Jin Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Stale CMS style from Halcyon (e.g. ${resource.xxx) needs to be edited.

Upcoming 2023

This needs to be modified.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should have the Negishi tag to allow better searching results.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Line 13 - "Negishi is being built" -> "Negishi was built"
Line 15 - "each 64-core Rome processor": I believe Negishi has Milan processors.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we add a [**Back to Negishi User Guide**](index.md) link at the bottom?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we add a [**Back to Negishi User Guide**](index.md) link at the bottom?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we add a Back to Negishi User Guide link at the bottom?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

boost: 2
---

# Creatting the Slurm Job Submission Script

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Typo: Creatting -> Creating

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Comment thread docs/userguides/negishi/software.md Outdated
```bash
$ module load biocontainers
```
More information on pre-downloaded ROCm containers can be found [here](https://biocontainer-doc.readthedocs.io/).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should be "pre-downloaded Biocontainers"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

```sinteractive``` accepts most of the same resource requests as ```sbatch```, so to request a login shell on the cpu account while allocating 2 nodes and 128 total cores, you might do:

```bash
sinteractive -A cpu -N2 -n256

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

cpu should be a partition not -A.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we need to discard this outdated page.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I removed the page.

@Guangzhen-Jin
Guangzhen-Jin merged commit 5507782 into main Jul 14, 2026
1 check 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