clulab / reach

Reach Biomedical Information Extraction
Other
96 stars 39 forks source link

Remove superfluous .par #756

Closed kwalcock closed 3 years ago

kwalcock commented 3 years ago

It looks like I didn't read the code I was replacing close enough. Here's a small correction.

kwalcock commented 3 years ago

Related to this, while testing fatdynet on the Mac, I found that the default .par (without any adjustments to the context and on the particular machine that does have more than 1 core) results in just one thread at a time. It wasn't until I added code like this and set the number of threads explicitly that parallel processing was achieved.

MihaiSurdeanu commented 3 years ago

Thanks @kwalcock !