Skip to content

Conversation

@tstenner
Copy link
Collaborator

@tstenner tstenner commented Aug 2, 2019

There have been some more bug reports with streams no / only one stream being found.

Currently, the "I'm looking for a stream"-multicast packets are only sent on one network adapter (on Linux it can be configured otherwise manually, but it ought to work out of the box).

So, this PR adds another config option for all local adapters multicast packets should be sent from, in case it's empty it's generated automatically (currently not yet working on Windows and not tested locally on macOS, but the previously failing CI is happy) and sends the multicast packets once for each configured adapter.

@tstenner tstenner added bug Something isn't working enhancement New feature or request labels Aug 2, 2019
@tstenner tstenner requested a review from cboulay August 2, 2019 12:46
@tstenner tstenner self-assigned this Aug 2, 2019
@tstenner tstenner force-pushed the multicast branch 26 times, most recently from 3905e98 to de09979 Compare August 7, 2019 07:38
@tstenner tstenner added this to the 1.15 milestone Nov 5, 2020
@tstenner tstenner force-pushed the multicast branch 2 times, most recently from 4c8b2bd to 5dc00d7 Compare February 24, 2021 11:10
@tstenner tstenner modified the milestones: 1.15, 1.16 Jun 21, 2021
@tstenner tstenner force-pushed the multicast branch 2 times, most recently from 92c79cc to aefcddb Compare September 28, 2021 06:32
@cboulay
Copy link
Collaborator

cboulay commented Sep 28, 2021

My only comment/suggestions is in the above review about where to combine the config-provided interfaces and the hardware-polled interfaces.

As far as I can tell, the reset of the code changes are related to storing addresses as ips instead of strings, and pre-emptive error handling. All good stuff.

@scott86
Copy link

scott86 commented Oct 15, 2021

@tstenner I just want to let you know that this PR is working great on our bridged lab systems (both linux). Previously we were using master and could not see any remote streams between the two.

@behinger
Copy link

hey! Trying to include this pull request in my linux installation (one direct LAN connection, one university LAN + wifi Connection, should look out for streams on the direct LAN and the wifi) - but so far no success.

  1. I checked out the pull request, and ran standalone_compilation_linux.sh.
  2. I copied the generated liblsl.so to pylsl/lib/ and installed it using pip install .
  3. I tried recording a stream using examples/ReceiveData.py -> but I cannot find anything.

I generally don't really know how to debug this issue, any pointers would be greatly appreciated!

@cboulay
Copy link
Collaborator

cboulay commented Dec 21, 2021

@behinger

I copied the generated liblsl.so to pylsl/lib/ and installed it using pip install .

Depending on the pylsl source version you're using, this might not do anything. Can you try putting your new liblsl.so somewhere it can be found AFTER you do pip install?

See the pylsl find_liblsl_libraries code. For your initial tests, the easiest is to set an environment variable PYLSL_LIB=path/to/custom/liblsl.so python myscript.py.

@behinger
Copy link

Thank you! I did something stupid 🙈 I checked out the PR but didnt switch to the branch. Now it works flawlessly! Super cool!

@tstenner tstenner changed the title [WIP] Send multicast requests on all adapters Send multicast requests on all adapters Feb 19, 2022
@tstenner tstenner marked this pull request as ready for review February 19, 2022 14:03
@tstenner tstenner closed this Feb 22, 2022
@tstenner tstenner deleted the multicast branch February 22, 2022 16:22
@behinger
Copy link

behinger commented Mar 1, 2022

is this PR actually pulled into the release? It appears in the releasenotes, but it didnt work for me. It is also not marked as pulled in this PR.

@tstenner tstenner restored the multicast branch March 1, 2022 13:15
@tstenner tstenner reopened this Mar 1, 2022
@tstenner tstenner merged commit c7a0f20 into sccn:master Mar 1, 2022
@tstenner tstenner deleted the multicast branch March 1, 2022 13:16
@tstenner
Copy link
Collaborator Author

tstenner commented Mar 1, 2022

No idea why it wasn't merged properly, but I have merged it manually (again).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants