Skip to content

Move docs into Notes; added screenshots; ReadMEz#6

Draft
NoWon1 wants to merge 1 commit into
ResearchNotesfrom
NoWon1-Notes-patch
Draft

Move docs into Notes; added screenshots; ReadMEz#6
NoWon1 wants to merge 1 commit into
ResearchNotesfrom
NoWon1-Notes-patch

Conversation

@NoWon1

@NoWon1 NoWon1 commented Jul 14, 2026

Copy link
Copy Markdown
Owner

Rename three documentation files into the Notes/ directory and apply a minor text formatting tweak in Notes/DeepResearch into the PJ by Gemz.txt. Add multiple new image assets under screenshot_data (including MMP reduced by half/ and OriGinal/ subfolders) to capture simulation outputs and plots.

@kagekageyamayama and @urswc
image

Rename three documentation files into the Notes/ directory and apply a minor text formatting tweak in Notes/DeepResearch into the PJ by Gemz.txt. Add multiple new image assets under screenshot_data (including MMP reduced by half/ and OG/ subfolders) to capture simulation outputs and plots.
@NoWon1 NoWon1 added documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Jul 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@NoWon1
NoWon1 marked this pull request as draft July 14, 2026 05:19
@NoWon1
NoWon1 requested a review from Copilot July 14, 2026 05:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@GonuDvc GonuDvc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Logic looks correct. One suggestion: the early return on line prevents the cleanup function from running — worth adding a finally block.

@NoWon1

NoWon1 commented Jul 15, 2026

Copy link
Copy Markdown
Owner Author

Logic looks correct. One suggestion: the early return on line prevents the cleanup function from running — worth adding a finally block.

Thanks for reviewing the logic. Will look into the function on my end. Best regards, Adi.

@copilot Kindly do the needful and lmk too.

Copilot AI commented Jul 15, 2026

Copy link
Copy Markdown

@NoWon1 I'm unable to start working on this because of repository rules that prevent me from pushing to the branch:

  • Changes must be made through a pull request due to repository rules

See the documentation for more details.

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

Labels

documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants