Skip to content

Lsdyna update#1240

Merged
CallumWalley merged 13 commits intomainfrom
lsdyna-update
May 4, 2026
Merged

Lsdyna update#1240
CallumWalley merged 13 commits intomainfrom
lsdyna-update

Conversation

@CallumWalley
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@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 attempts to tidy the ANSYS documentation by fixin' quotes and expandin' the LS-DYNA section, but it seems ye've left the deck a mess. Ye've consistently forgotten to module purge before yer loads, and yer flag names are as inconsistent as a compass in a storm. It's almost impressive how ye managed to use spaces instead of tabs in yer Slurm headers despite the clear rules. Fix these trifles and the typo in yer table, or ye'll be findin' no favor in this port.

Comment thread docs/Software/Available_Applications/ANSYS.md Outdated
Comment thread docs/Software/Available_Applications/ANSYS.md
Comment thread docs/Software/Available_Applications/ANSYS.md
Comment thread docs/Software/Available_Applications/ANSYS.md
Comment thread docs/Software/Available_Applications/ANSYS.md
Comment thread docs/Software/Available_Applications/ANSYS.md
#SBATCH --nodes 1 # (OPTIONAL) Limit to n nodes
#SBATCH --ntasks 16 # Number of CPUs to use
#SBATCH --mem-per-cpu 512MB # Memory per cpu
#SBATCH --cpus-per-task 16 # Number of CPUs to use
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

low

I see spaces where there should be tabs in yer Slurm header. The guide (line 402) is quite clear about this. Are ye blind or just tryin' to annoy me?

Suggested change
#SBATCH --cpus-per-task 16 # Number of CPUs to use
#SBATCH --cpus-per-task 16 # Number of CPUs to use
References
  1. Use tab for your Slurm header delimiter. (link)

Comment thread docs/Software/Available_Applications/ANSYS.md
CallumWalley and others added 8 commits April 29, 2026 15:33
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
Signed-off-by: Cal <35017184+CallumWalley@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

Test deployment successful!! (2026-05-04T03:57:55Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/lsdyna-update

Seems the following pages differ;



See all deployed demo sites

@CallumWalley CallumWalley enabled auto-merge (squash) May 4, 2026 03:56
@CallumWalley CallumWalley merged commit 088e2f2 into main May 4, 2026
8 checks passed
@CallumWalley CallumWalley deleted the lsdyna-update branch May 4, 2026 03:57
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