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

Osier chapter #18

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Osier chapter #18

wants to merge 25 commits into from

Conversation

samgdotson
Copy link
Collaborator

@samgdotson samgdotson commented Jan 21, 2025

@samgdotson samgdotson added Difficulty:3-Expert This issue is not only complicated, but also complex. It may require special advanced 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) Writing:Chapter This issue has to do with reviewing or writing a thesis chapter. labels Jan 21, 2025
@samgdotson samgdotson added this to the Full draft of thesis milestone Jan 21, 2025
@samgdotson samgdotson requested review from katyhuff and munkm January 21, 2025 20:59
@samgdotson samgdotson self-assigned this Jan 21, 2025
@pep8speaks
Copy link

pep8speaks commented Jan 21, 2025

Hello @samgdotson! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 13:80: E501 line too long (81 > 79 characters)

Line 48:80: E501 line too long (80 > 79 characters)

Comment last updated at 2025-01-21 22:21:58 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty:3-Expert This issue is not only complicated, but also complex. It may require special advanced 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) Writing:Chapter This issue has to do with reviewing or writing a thesis chapter.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chapter Review - Madicken] Osier chapter [Chapter 3] Osier chapter [Chapter Review - Katy] Osier chapter
2 participants