Skip to content

[heavy_tails] update urls for datasets used #191

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

Merged
merged 1 commit into from
May 4, 2023
Merged

Conversation

shlff
Copy link
Member

@shlff shlff commented May 3, 2023

Hi @jstac this PR updates the dataset urls used in the lecture heavy_tails after merging the corresponding PR 3 in repo high_dim_data.

@netlify
Copy link

netlify bot commented May 3, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 71c357c
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/645214c2d5828900081cc6aa
😎 Deploy Preview https://deploy-preview-191--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented May 3, 2023

@jstac
Copy link
Contributor

jstac commented May 3, 2023

Thanks @shlff . Please let me know when you want me to merge.

@shlff
Copy link
Member Author

shlff commented May 4, 2023

Thanks @jstac . I've confirmed that the figures are generated well with the replaced URLs. Please find here:

This PR is ready for review.

@Smit-create
Copy link
Member

Thanks @shlff. @jstac, Should we merge this and try to rebase #192 on top of this?

@jstac
Copy link
Contributor

jstac commented May 4, 2023

Thanks @shlff , @Smit-create . Merging.

@jstac jstac merged commit 600eff4 into main May 4, 2023
@jstac jstac deleted the update_heavy_tails branch May 4, 2023 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants