Skip to content

Add simpler fix for viewing plexos output#863

Open
harrysorensennrel wants to merge 2 commits intoNatLabRockies:gh-pagesfrom
harrysorensennrel:gh-pages
Open

Add simpler fix for viewing plexos output#863
harrysorensennrel wants to merge 2 commits intoNatLabRockies:gh-pagesfrom
harrysorensennrel:gh-pages

Conversation

@harrysorensennrel
Copy link
Copy Markdown
Collaborator

Use the iconv command to allow viewing of plexos outputs.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Plexos run documentation to provide a simpler method for viewing SLURM output by converting it to UTF-8 with iconv.

Changes:

  • Replaces the Vim-based “remove special characters” guidance with an iconv conversion approach.
  • Updates the note explaining why SLURM output may display incorrectly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/Documentation/Applications/Plexos/run_plexos.md Outdated
Comment on lines 60 to 62
```
# On PC
:%s/<CTRL-2>//g
# On Mac
%s/<CTRL-SHIFT-2>//g
iconv -f UTF-16 -t UTF-8 your_output_file.txt
```
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

The iconv example currently just writes the converted content to stdout; readers may expect a converted file. Consider showing either an explicit output (via -o or shell redirection) or piping to a pager (e.g., | less) since the stated goal is “viewing” the output. Also, other code blocks in this doc use language tags (e.g., bash), so add one here for consistency.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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