Skip to content

@schedule is deprecated, the code is no longer valid in 1.0#28800

Merged
KristofferC merged 2 commits intoJuliaLang:masterfrom
iht:fix_schedule_doc
Aug 23, 2018
Merged

@schedule is deprecated, the code is no longer valid in 1.0#28800
KristofferC merged 2 commits intoJuliaLang:masterfrom
iht:fix_schedule_doc

Conversation

@iht
Copy link
Copy Markdown
Contributor

@iht iht commented Aug 21, 2018

I am following the documentation to learn Julia, and I found a macro, @schedule that is deprecated in Julia 1.0, according to #27164

This pull request simply replaces @schedule by @async in the Parallel Computing section of the manual.

@garrison garrison added the docs This change adds or pertains to documentation label Aug 21, 2018

julia> for i in 1:4 # start 4 tasks to process requests in parallel
@schedule do_work()
@async do_work()
Copy link
Copy Markdown
Member

@KristofferC KristofferC Aug 22, 2018

Choose a reason for hiding this comment

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

round(exec_time,2) a few lines below doesn't work on 1.0 should be round(exec_time; digits=2)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also, the n = n - 1 needs a global in front of it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have pushed a new commit with those changes. Thanks for the comments, nice catch!

@KristofferC KristofferC merged commit 6d4ff5e into JuliaLang:master Aug 23, 2018
KristofferC pushed a commit that referenced this pull request Aug 23, 2018
* @schedule is deprecated, the code is no longer valid in 1.0

* Additional changes for 1.0 compliance

(cherry picked from commit 6d4ff5e)
@iht iht deleted the fix_schedule_doc branch August 23, 2018 13:02
staticfloat pushed a commit that referenced this pull request Aug 24, 2018
* @schedule is deprecated, the code is no longer valid in 1.0

* Additional changes for 1.0 compliance
KristofferC pushed a commit that referenced this pull request Sep 8, 2018
* @schedule is deprecated, the code is no longer valid in 1.0

* Additional changes for 1.0 compliance

(cherry picked from commit 6d4ff5e)
KristofferC pushed a commit that referenced this pull request Sep 8, 2018
* @schedule is deprecated, the code is no longer valid in 1.0

* Additional changes for 1.0 compliance

(cherry picked from commit 6d4ff5e)
KristofferC pushed a commit that referenced this pull request Feb 11, 2019
* @schedule is deprecated, the code is no longer valid in 1.0

* Additional changes for 1.0 compliance

(cherry picked from commit 6d4ff5e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs This change adds or pertains to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants