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

Update XGBoost tutorial #9

Merged
merged 19 commits into from
Aug 14, 2024
Merged

Update XGBoost tutorial #9

merged 19 commits into from
Aug 14, 2024

Conversation

spestana
Copy link
Contributor

Hi @savalann , I've gone ahead and fixed a few things in these notebooks from your initial pull request:

  • added notebooks to the table of contents
  • added an "introduction" page for the text at the beginning of the tutorial
  • updated the conda environment so that the notebook runs and is rendered on the website
  • removed a user input line which would prevent the notebook from being rendered on the website
  • fixed some typos ... and hopefully did not introduce any new ones! ;)

@spestana spestana requested a review from savalann August 14, 2024 21:41
@spestana
Copy link
Contributor Author

@savalann , can we merge this PR, and then you can make any additional updates afterwards?

@spestana spestana added the preview Create a website PR preview label Aug 14, 2024
@spestana spestana mentioned this pull request Aug 14, 2024
Merged
@github-actions GitHub Actions
Copy link

🚀 Deployed on https://deploy-preview-9--geosmart-website-2024.netlify.app

@spestana spestana merged commit 884c4f2 into geo-smart:main Aug 14, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@savalann savalann Awaiting requested review from savalann

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

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