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

Projects - Add list with link to repositories #30

Merged
merged 4 commits into from
Aug 25, 2024
Merged

Projects - Add list with link to repositories #30

merged 4 commits into from
Aug 25, 2024

Conversation

jomey
Copy link
Member

@jomey jomey commented Aug 22, 2024

I went the simple route and only listed the projects with their links. All the other details are on each page.

@jomey jomey added the preview Create a website preview label Aug 22, 2024
@jomey jomey requested a review from spestana August 22, 2024 16:35
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 22, 2024 •
edited
Loading

🚀 Deployed on https://deploy-preview-30--snowex-website-2024.netlify.app

@jomey
Copy link
Member Author

jomey commented Aug 22, 2024

The failure on OS X is due to installing pytorch in one of the tutorials. I accepted that as a 'fix later' item

spestana
spestana reviewed Aug 23, 2024
View reviewed changes
Copy link
Member

@spestana spestana left a comment

Choose a reason for hiding this comment

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

link syntax 🙃

book/projects/index.md Outdated Show resolved Hide resolved
book/projects/index.md Outdated Show resolved Hide resolved
book/projects/index.md Outdated Show resolved Hide resolved
book/projects/index.md Outdated Show resolved Hide resolved
book/projects/index.md Outdated Show resolved Hide resolved
book/projects/index.md Outdated Show resolved Hide resolved
aaarendt
aaarendt approved these changes Aug 23, 2024
View reviewed changes
Copy link
Member

@aaarendt aaarendt left a comment

Choose a reason for hiding this comment

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

looks good now! Should we wait to add any other project teams?

book/projects/index.md Outdated Show resolved Hide resolved
@jomey
Copy link
Member Author

jomey commented Aug 23, 2024

looks good now! Should we wait to add any other project teams?

I used the ones from this organization that had commit activity

@jomey
Copy link
Member Author

jomey commented Aug 23, 2024

The build is currently failing because of issues pulling data for SNOTEL

@scottyhq
Copy link
Member

The build is currently failing because of issues pulling data for SNOTEL

An option is to change the execution setting at this point to 'auto'... although I don't think they're currently cached in CI...

website-2024/book/_config.yml

Line 34 in 039c4b6

execute_notebooks: 'force'

@jomey
Copy link
Member Author

jomey commented Aug 23, 2024

The build is currently failing because of issues pulling data for SNOTEL

An option is to change the execution setting at this point to 'auto'... although I don't think they're currently cached in CI...

website-2024/book/_config.yml

Line 34 in 039c4b6

execute_notebooks: 'force'

Yeah, we didn't cache it before and we would need at least one successful run

@jomey
Copy link
Member Author

jomey commented Aug 25, 2024

Closes #26

@jomey
Book config - Start using the Jupyter build cache
c7e5e45 
Trim down the build time for smaller book edits and start using the Jupyter cache.
@jomey jomey merged commit 060dabd into main Aug 25, 2024
3 of 4 checks passed
@jomey jomey deleted the projects branch August 25, 2024 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spestana spestana spestana left review comments

@aaarendt aaarendt aaarendt approved these changes

@scottyhq scottyhq Awaiting requested review from scottyhq

@JessicaS11 JessicaS11 Awaiting requested review from JessicaS11

Assignees
No one assigned
Labels
preview Create a website preview
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jomey @scottyhq @spestana @aaarendt