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

Multi user support #21

Closed
FabianRig opened this issue Jan 16, 2024 · 14 comments
Closed

Multi user support #21

FabianRig opened this issue Jan 16, 2024 · 14 comments

Comments

@FabianRig
Copy link
Owner

TODO: Include changes to library so that multi user support is possible.

See: #19 (comment)

@FabianRig
Copy link
Owner Author

I've included a fork of the original BLE serial library in 07d139b. That should hopefully enable multi user support.

If further testing shows that everything is working correctly, I'll update the README accordingly.

@Aiasa21
Copy link

Aiasa21 commented Feb 2, 2024

Hi FabianRig, I'm using your code sussefully on my home moonboard. I don't understand where I can find the last code that implement the multi user support.
Thank you for the help!

@FabianRig
Copy link
Owner Author

There are no code changes in the project but in the dependencies. Have a look at https://github.com/FabianRig/ArduinoMoonBoardLED/blob/main/platformio.ini where you can see that one dependency has been replaced with a custom version (forked and patched by me).

Just make sure you have the most recent version of this repo (git pull or git clone to a new directory) and that's it.

@Aiasa21
Copy link

Aiasa21 commented Feb 3, 2024

Hi, I add the new lines in the HardwareBLESerial.cpp and I'm just trying to use two moonboard apps at the same time and it works properly, great job guys!

@FabianRig
Copy link
Owner Author

Good to hear that it's working for you. But that's not the intended way. You can simply clone this repo -- no manual adjustments needed!

@Aiasa21
Copy link

Aiasa21 commented Feb 3, 2024

I'm sorry about that, actually I don't know how to do so yesterday I prepared the edit manually.
As soon as I have some minutes I'll try to figure out how to do it and update directly in the visual studio project so to give you a feedback.

@scott-lin
Copy link

Should the hardware solution for multi-user support be removed now that you have added a software solution? Namely, the readme updates from commit 3fd616a.

@FabianRig
Copy link
Owner Author

I'm sorry about that, actually I don't know how to do so yesterday I prepared the edit manually. As soon as I have some minutes I'll try to figure out how to do it and update directly in the visual studio project so to give you a feedback.

No need to apologize. :-) It's just more work to manually change code when you can just git clone / git pull. If you want a simple solution: just download this repo again to another folder and build and flash the firmware again.

@FabianRig
Copy link
Owner Author

Should the hardware solution for multi-user support be removed now that you have added a software solution? Namely, the readme updates from commit 3fd616a.

You're absolutely right. I changed the README.

@scott-lin
Copy link

Just wanted to drop by again and say thanks for this particular update! It works great, and my wife and I are enjoying it.

@FabianRig
Copy link
Owner Author

Thank you for your kind words! :-) I'm glad it's working for you!

@Aiasa21
Copy link

Aiasa21 commented Feb 13, 2024

I'm sorry about that, actually I don't know how to do so yesterday I prepared the edit manually. As soon as I have some minutes I'll try to figure out how to do it and update directly in the visual studio project so to give you a feedback.

No need to apologize. :-) It's just more work to manually change code when you can just git clone / git pull. If you want a simple solution: just download this repo again to another folder and build and flash the firmware again.

I did this and I confirm that it work good, great job.. Yesterday we climbed with 4 app simultaneously, amazing!

Just to know, there's a maximum connection limit?

@FabianRig
Copy link
Owner Author

Nice, that's good to know! To be honest: I assume that there is a limit, but I don't know what the limit is. There is a limit for concurrent connections in the underlying libraries. However, the BLE Serial is not really a "connection" in that sense, so I'm not sure whether that internal limit applies.

Four parallel connections is already impressive. :-D We'll have to round up as many devices as possible and try it out. ;-)

@Aiasa21
Copy link

Aiasa21 commented Feb 13, 2024

I understand, maybe I could try with 5 connection if one day we'll climb together, I'll let you know if this will happen. Anyway I suggest to disable the setting "Light problem on swipe" otherwise is a mess :D

I would like to suggest one update:
it could be possible in the future to set a lower brightness during the start cycling so as to avoid an excessive use of current and then turn the brightness high during the climb?
That's because the section of the wire used for the led is extremely low and while all the leds are on the current is really high.

Thanks again for your work!

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

No branches or pull requests

3 participants