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

Update to SDM workflow example in the docs #831

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

tiemvanderdeure
Copy link
Contributor

Uses SDM.jl/MLJ to do the whole workflow.

Also updated a few steps that are a lot less clunky now.

(And replaced plots.jl with makie, but feel free to reverse/improve).

@rafaqz
Copy link
Owner

rafaqz commented Dec 6, 2024

Thanks, looks good

@rafaqz rafaqz merged commit b9379e1 into rafaqz:main Dec 6, 2024
1 of 2 checks passed
@tiemvanderdeure
Copy link
Contributor Author

Hm I just took a look at the docs and the output of the last cells isn't shown - and it has @example in the top right. I must have made some typo?
billede

@rafaqz
Copy link
Owner

rafaqz commented Dec 7, 2024

Huh, I can't see anything obvious

@tiemvanderdeure
Copy link
Contributor Author

Same, and it doesn't error either. It could be something with the SDM package as it's unregistered.

@rafaqz
Copy link
Owner

rafaqz commented Dec 7, 2024

That shouldn't make a difference. I would guess a missing newline or something.

But also that package really wants registering ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants