Skip to content

Docs: Generalize with Tabs - #468

Open
ax3l wants to merge 11 commits into
BLAST-AI-ML:mainfrom
ax3l:docs-general-tabs
Open

ax3l wants to merge 11 commits into
BLAST-AI-ML:mainfrom
ax3l:docs-general-tabs

Conversation

@ax3l

@ax3l ax3l commented Sep 18, 2026 •

Copy link
Copy Markdown
Member

Generalize our docs to be "General Pattern" + "Example Project: ..." style.

That way we:

  • keep a fully working, copy-pastable version
  • have by default a pattern that people can adopt to their own projects

Follow-up to #341

@ax3l
ax3l requested a review from EZoni September 18, 2026 17:47
- The generic dashboard-training guidance misses the hard-coded realtime QoS
- The getting-started tunnel is incompatible with training when MongoDB uses a non-default port
- A remaining BELLA-specific command is both ungeneralized and invalid
Comment thread docs/source/dashboard.md Outdated
Comment thread docs/source/dashboard.md Outdated
Comment thread docs/source/dashboard.md

3. Set up the database settings (read-only) and the AmSC MLflow API key:
```bash
export SF_DB_HOST='127.0.0.1'

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.

Removed: went away in a recent PR.
We still need to remove references to it in a few other locations: https://github.com/search?q=repo%3ABLAST-AI-ML%2Fsynapse+SF_DB_HOST&type=code

Comment thread docs/source/getting-started.md
Comment thread docs/source/getting-started.md
Comment thread docs/source/getting-started.md
Comment thread docs/source/getting-started.md
Comment thread docs/source/ml-training.md
Comment thread docs/source/deployment.md Outdated
Comment thread docs/source/dashboard.md Outdated
Comment thread docs/source/deployment.md Outdated
Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>
@ax3l

ax3l commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

LGTM for a major improvement. Would merge and improve small nits further in separate PRs.

@ax3l
ax3l requested review from EZoni and RemiLehe September 22, 2026 20:54
```

```{note}
This path is hardcoded for the BELLA deployment in {repo}`dashboard/parameters_manager.py`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
This path is hardcoded for the BELLA deployment in {repo}`dashboard/parameters_manager.py`.
This path is hardcoded for the BELLA deployment in {repo}`dashboard/parameters_manager.py` and {repo}`dashboard/model_manager.py`.

Comment on lines -161 to +301
podman-hpc run --gpu -v /etc/localtime:/etc/localtime -v $HOME/db.profile:/root/db.profile -v /path/to/config.yaml:/app/ml/config.yaml --rm -it registry.nersc.gov/m558/superfacility/synapse-ml:latest python -u /app/ml/train_model.py --test --config_file /app/ml/config.yaml --model NN
podman-hpc run --gpu -v /etc/localtime:/etc/localtime --env-file $HOME/db-podman.profile -v <your_config_file>:/app/ml/config.yaml --rm -it registry.nersc.gov/m558/superfacility/synapse-ml:latest python -u /app/ml/train_model.py --test --config_file /app/ml/config.yaml --model NN

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just double checking, these changes in the code snippet,

  • from db.profile to db-podman.profile
  • from -v to --env-file

happen to fix issues that we had missed so far?

conda activate base
conda-lock install --name synapse-gui environment-lock.yml
conda activate synapse-gui
export SF_DB_READONLY_PASSWORD='...'

@EZoni EZoni Sep 22, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export SF_DB_READONLY_PASSWORD='...'
export SF_DB_READONLY_PASSWORD='<your_password_here>' # Use SINGLE quotes around the password!

conda activate base
conda-lock install --name synapse-ml environment-lock.yml
conda activate synapse-ml
export SF_DB_READONLY_PASSWORD='...'

@EZoni EZoni Sep 22, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export SF_DB_READONLY_PASSWORD='...'
export SF_DB_READONLY_PASSWORD='<your_password_here>' # Use SINGLE quotes around the password!

:sync: bella-nersc

```text
SF_DB_READONLY_PASSWORD=your_password_here

@EZoni EZoni Sep 22, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
SF_DB_READONLY_PASSWORD=your_password_here
SF_DB_READONLY_PASSWORD=<your_password_here>

Comment thread docs/source/dashboard.md
3. Set up the database settings (read-only) and the AmSC MLflow API key:
```bash
export SF_DB_HOST='127.0.0.1'
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!
export SF_DB_READONLY_PASSWORD='<your_password_here>' # Use SINGLE quotes around the password!

Comment thread docs/source/dashboard.md
```bash
export SF_DB_HOST='127.0.0.1'
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!
export AM_SC_API_KEY='your_amsc_api_key_here' # Required when MLflow tracking_uri is AmSC

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export AM_SC_API_KEY='your_amsc_api_key_here' # Required when MLflow tracking_uri is AmSC
export AM_SC_API_KEY='<your_amsc_api_key_here>' # Required when MLflow tracking_uri is AmSC


3. Set up the database settings (read-only) and the AmSC MLflow API key:
```bash
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!
export SF_DB_READONLY_PASSWORD='<your_password_here>' # Use SINGLE quotes around the password!

:sync: bella-nersc

```bash
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export SF_DB_READONLY_PASSWORD='your_password_here' # Use SINGLE quotes around the password!
export SF_DB_READONLY_PASSWORD='<your_password_here>' # Use SINGLE quotes around the password!

Comment thread docs/source/dashboard.md
4. Run the Docker container:
```bash
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_HOST='127.0.0.1' -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' synapse-gui
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' synapse-gui

@EZoni EZoni Sep 22, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' synapse-gui
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='<your_password_here>' -e AM_SC_API_KEY='<your_amsc_api_key_here>' synapse-gui

Comment thread docs/source/dashboard.md
For debugging, you can enter the container without starting the app:
```bash
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_HOST='127.0.0.1' -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' -it synapse-gui bash
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' -it synapse-gui bash

@EZoni EZoni Sep 22, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='your_password_here' -e AM_SC_API_KEY='your_amsc_api_key_here' -it synapse-gui bash
docker run --network=host -v /etc/localtime:/etc/localtime -v $PWD/ml:/app/ml -e SF_DB_READONLY_PASSWORD='<your_password_here>' -e AM_SC_API_KEY='<your_amsc_api_key_here>' -it synapse-gui bash

export SF_DB_HOST='127.0.0.1'
export SF_DB_READONLY_PASSWORD='...'
export AM_SC_API_KEY='...'
export <database.password_ro_env>='...'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
export <database.password_ro_env>='...'
export <database.password_ro_env>='<your_password_here>' # Use SINGLE quotes around the password!

@ax3l

ax3l commented Oct 6, 2026

Copy link
Copy Markdown
Member Author

@EZoni do you like this finish this PR this week (do not over-do it) and we can merge a first generalized version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants