Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve documentation with Argo/GPT4o for data management #569

Merged
merged 15 commits into from
Jan 11, 2025

Conversation

keceli
Copy link
Contributor

@keceli keceli commented Dec 9, 2024

Similar to earlier PRs (#562, #565), using Argo/GPT4o to improve the documentation for data management. See issue #563.

keceli and others added 12 commits December 8, 2024 18:41
   - Changed "Command line" to "Command-line" for consistency.
   - Changed "web-based" to lowercase for consistency.
   - Added commas for clarity in lists and complex sentences.
2. Fixed formatting issues:
   - Removed unnecessary backslashes in paths.
   - Added a line break after the "Getting Started" URL for better readability.
3. Clarified language:
   - Specified "SSH" in uppercase as it is an acronym.
   - Clarified the use of paths by removing unnecessary escape characters.
…ocal transfers of small files" for clarity.

2. Improved formatting: No changes were necessary as the formatting is already clear and concise.
…ttps://www.globus.org).This" to separate sentences.

2. Fixed grammar: Added a period at the end of the sentence in "Django Globus Portal Framework customized for a variety of different projects" for completeness.
3. Fixed grammar: Added a period at the end of the sentence in "users can access ACDC using their campus or institution federated username and passcode" for completeness.
…collection" and ensuring consistent use of terms like "guest collection" and "Globus".

2. Fixed formatting issues, such as ensuring consistent use of bullet points and headings.
3. Clarified instructions and descriptions for better readability and understanding.
4. Removed redundant or unclear statements to improve clarity.
5. Ensured consistent use of terminology and formatting throughout the document.
…d correcting verb forms.

2. Fixed formatting issues, including consistent use of punctuation and capitalization.
3. Clarified instructions and ensured consistency in the use of terms like "ALCF account" and "Eagle filesystem."
4. Added missing periods at the end of some bullet points for consistency.
5. Ensured that the use of code formatting for filenames and scripts is consistent throughout the document.
… punctuation for clarity.

2. Improved formatting: Added bash syntax highlighting for code blocks to enhance readability.
3. Clarified instructions: Added missing backticks around commands and corrected the phrasing for consistency.
4. Fixed broken link placeholder: Added a placeholder link for "Using Globus" to indicate where a link should be provided.
…d consistency.

2. Fixed formatting issues, including ensuring consistent use of spaces and punctuation.
3. Corrected the spelling of "collaborators" and "capabilities."
4. Added missing periods at the end of some sentences for consistency.
5. Clarified instructions for disabling the Transfer Agent in HSI.
6. Added a missing link reference for "Using Globus" to ensure the link is functional.
…or clarity.

2. Improved formatting: Changed "Following are descriptions" to "Below are descriptions" for better readability.
3. Added bash syntax highlighting to the command examples for consistency and readability.
4. Corrected a repeated acronym: "ALCC/ALCC" to "ALCC/ESP" in the Quota Increases section.
…r punctuation.

2. Corrected the formatting of the table under "Polaris SSD Specs" to ensure proper alignment.
3. Added bash syntax highlighting to code blocks for better readability.
4. Clarified some sentences for improved understanding, such as specifying paths with backticks for consistency.
@felker felker self-requested a review January 11, 2025 00:06
Copy link
Member

@felker felker left a comment

Choose a reason for hiding this comment

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

The LLM likes to remove commented-out text <!-- ..., which might not always be a desired outcome, e.g. throughout docs/data-management/acdc/eagle-data-sharing.md, it removed these HTML comments, including for example:

<!--- There are two ways for a PI to access their project directory on Eagle. 
1. **Web Interface:** By logging in to Globus interface directly and navigating to the ALCF Eagle endpoint. -->
...
<!--- 
<figure markdown>
  ![File Manager](files/file-manager.png){ width="700" }
  <figcaption>File Manager</figcaption>
</figure>

2. **POSIX:** By logging in to the ALCF systems from the terminal window.
<figure markdown>
  ![Terminal Window](files/terminal-window.png){ width="700" }
  <figcaption>Terminal Window</figcaption>
</figure> --->

just something to note for now; I let it remove them

@felker felker merged commit 6b81d4c into main Jan 11, 2025
1 check passed
@felker felker deleted the keceli_argo_fix_data branch January 11, 2025 00:09
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