Skip to content

Explicitly use 'fork' method for multiprocessing in io.ascii - #8852

Closed
astrofrog wants to merge 2 commits into
astropy:masterfrom
astrofrog:default-fork-method
Closed

astrofrog wants to merge 2 commits into
astropy:masterfrom
astrofrog:default-fork-method

Conversation

@astrofrog

Copy link
Copy Markdown
Member

This is a workaround for the issue described in #8851 - ideally we would fix the underlying cause, but this will at least keep things working once Python 3.8 if we don't.

This will need a changelog entry but I'll wait until the 2.0.15 sections are created in the changelog.

@codecov

codecov Bot commented Jun 15, 2019 •

Copy link
Copy Markdown

Codecov Report

Merging #8852 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8852   +/-   ##
=======================================
  Coverage   86.99%   86.99%           
=======================================
  Files         400      400           
  Lines       59480    59480           
  Branches     1100     1100           
=======================================
  Hits        51745    51745           
  Misses       7094     7094           
  Partials      641      641

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cfe7616...f1c203e. Read the comment docs.

@astropy astropy deleted a comment from codecov Bot Jun 15, 2019
…ludes a change to 'spawn', for which the fast reader doesn't work.
@astrofrog
astrofrog force-pushed the default-fork-method branch from f5720df to f1c203e Compare June 15, 2019 18:24
@astrofrog
astrofrog requested review from abonaca and taldcroft and removed request for abonaca June 15, 2019 19:20
@astrofrog

Copy link
Copy Markdown
Member Author

Maybe we can wait a little to see if the rest of the fixes for #8853 can be identified before merging this.

@astrofrog

Copy link
Copy Markdown
Member Author

I fixed the issues with 'spawn' in #8853 so closing this since the workaround is no longer needed.

@astrofrog astrofrog closed this Jun 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant