Skip to content
This repository has been archived by the owner on Dec 11, 2022. It is now read-only.

Sync workers with global lock #361

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kevinzou1982
Copy link

compare to my last pull request
1, use global sync lock instand of lock_counter and release_counter in tensorflow. speed up counter access and prevent not zero problem.
2, fix a sync bug. Clipped_ppo_agent for example
if int(batch.size / self.ap.network_wrappers['main'].batch_size) is different between each workers, some worker will be in acting phase and other workers will be block in wait_for_all_workers_barrier

kevinzou1982 and others added 2 commits June 20, 2019 09:51
…n tensorflow. speed up counter access and prevent not zero problem.

2, fix a sync bug. Clipped_ppo_agent for example
if int(batch.size / self.ap.network_wrappers['main'].batch_size) is different between each workers, some worker will be in acting phase and other workers will be block in wait_for_all_workers_barrier
@gal-leibovich
Copy link
Contributor

is this replacing #347?

@kevinzou1982
Copy link
Author

Yes for my prj

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

Successfully merging this pull request may close these issues.

2 participants