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

Threaded distributed MAPF #20

Closed
wants to merge 1 commit into from
Closed

Threaded distributed MAPF #20

wants to merge 1 commit into from

Conversation

guilyx
Copy link
Member

@guilyx guilyx commented Jan 27, 2021

  • Distributed MAPF means each robot work on its own
  • Threads make sense to replicate that
  • /!\ Shared data among dictionaries seems to be corrupted, even with Manager().
    • Idea: use locks

Closes #17

@guilyx guilyx added the enhancement New feature or request label Jan 27, 2021
@guilyx guilyx self-assigned this Jan 27, 2021
@codecov-io
Copy link

codecov-io commented Jan 27, 2021

Codecov Report

Merging #20 (b8fef81) into main (f7dbafa) will decrease coverage by 1.29%.
The diff coverage is 13.79%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   42.69%   41.39%   -1.30%     
==========================================
  Files          12       12              
  Lines         534      558      +24     
==========================================
+ Hits          228      231       +3     
- Misses        306      327      +21     
Impacted Files Coverage Δ
pymapf/decentralized/nmpc.py 35.07% <13.79%> (-4.93%) ⬇️

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 f7dbafa...b8fef81. Read the comment docs.

@guilyx guilyx closed this Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Concurrency
2 participants