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.

Sign up for GitHub

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

Jump to bottom

adds sgd changes #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

adds sgd changes #1

wants to merge 1 commit into from

Conversation

samgdotson
Copy link
Collaborator

@samgdotson samgdotson commented Jan 24, 2024 •
edited
Loading

@rileyf2 please see these changes. I don't think @munkm added you to the organization, yet, so I can't make you a reviewer.

Also, if you want to perhaps work in a more streamlined manner to accelerate this project please see Overleaf (google docs for latex).

What I need from @rileyf2

  • Add the remaining documents to the Zotero with parent items and (reasonably) complete metadata.
  • Comment on the changes I made -- preferably here, but I will also accept Overleaf for the moment.
  • "Fork" the ARFC repository to your personal github account (as you did with the group website).
  • Ask me or @munkm for help (it is better to ask early and often than try to reinvent the wheel).
  • Review the "writing checklist"

Regarding that last item, I'm sure other ARFC members would be willing to help as well.

@samgdotson samgdotson added Comp:Analysis This issue has to do with the analysis component of the code or document. (plots, postprocessing) Difficulty:2-Challenging This issue may be complex or require specialized skills. Priority:1-Critical This is the highest priority (i.e. it is blocking other work or facing a deadline). Status:5-In Review This issue has been handled, and the solution is being reviewed. (probably as a PR) labels Jan 24, 2024
@samgdotson samgdotson requested a review from munkm January 24, 2024 17:37
@samgdotson samgdotson self-assigned this Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@munkm munkm Awaiting requested review from munkm

At least 1 approving review is required to merge this pull request.

Assignees

@samgdotson samgdotson

Labels
Comp:Analysis This issue has to do with the analysis component of the code or document. (plots, postprocessing) Difficulty:2-Challenging This issue may be complex or require specialized skills. Priority:1-Critical This is the highest priority (i.e. it is blocking other work or facing a deadline). Status:5-In Review This issue has been handled, and the solution is being reviewed. (probably as a PR)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@samgdotson