Skip to content

Conversation

@pierreluctg
Copy link
Collaborator

When neoVI server is enabled, there is an issue with concurrent device open.

@pierreluctg pierreluctg force-pushed the neovi-device-open-mutex branch from 799cf02 to ac9414c Compare June 11, 2019 12:30
@felixdivo
Copy link
Collaborator

Maybe we could make os.path.join(tempfile.gettempdir(), "neovi.lock") a direct member of the module so it can be modified?

@codecov
Copy link

codecov bot commented Jun 11, 2019

Codecov Report

Merging #621 into develop will decrease coverage by 0.09%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##           develop     #621     +/-   ##
==========================================
- Coverage       64%   63.91%   -0.1%     
==========================================
  Files           63       63             
  Lines         5607     5623     +16     
==========================================
+ Hits          3589     3594      +5     
- Misses        2018     2029     +11

@pierreluctg pierreluctg force-pushed the neovi-device-open-mutex branch from 4e2e76a to 3987593 Compare June 11, 2019 17:48
@pierreluctg pierreluctg force-pushed the neovi-device-open-mutex branch from 3987593 to e18c2d5 Compare June 11, 2019 18:09
@felixdivo felixdivo merged commit 833c4f5 into develop Jun 12, 2019
@felixdivo felixdivo deleted the neovi-device-open-mutex branch June 12, 2019 14:48
@pierreluctg pierreluctg mentioned this pull request Aug 25, 2020
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.

4 participants