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.

Sign up for GitHub

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

Jump to bottom

Develop #16

Merged
merged 153 commits into from
Dec 20, 2021
Merged

Develop #16

merged 153 commits into from
Dec 20, 2021

Conversation

supertoto29
Copy link
Contributor

Warning : Before creating a pull request, make sure that it does not already exists in the pull requests. Thank you.

Description

Add backup of the project as the school project is done

How has this been tested ?

Tested locally with the sensors

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

supertoto29 and others added 30 commits September 2, 2021 21:03
@supertoto29
add files and start node
c37bbc3
@supertoto29
add files start node
7ae9665
@supertoto29
fix for compilation
05d78ad
@supertoto29
fix constructor
aa717e2
@supertoto29
update package.xml
1d673ed
@supertoto29
version bump
1b1d50f
@supertoto29
fix compilation run_depend to exec_depend
dc73c20
@supertoto29
change devcontainer.json
e3d1285
@supertoto29
add serial connection
100aba5
@supertoto29
change private variables
4d38f8b
@supertoto29
change order for initialization
0b00cb7
@supertoto29
test of node
97e9744
@supertoto29
from unneed files
64a3ced
@supertoto29
change for launch
f44c33a
@supertoto29
remove bootstrap
b506477
@supertoto29
starting the node
5f12872
@supertoto29
add checksum same as IMU
1221b57
@supertoto29
start transmit
4140b7b
@supertoto29
fix condition
bf06352
@supertoto29
Merge pull request #1 from sonia-auv/feature/init-node
052e4ef 
Feature/init node
@supertoto29
start working with the sensor
d559c35
@supertoto29
add license for serial.cc and serial.h
d9f6272
@supertoto29
change path for usb to debug
193739a
@supertoto29
fix dialout for sonia user in the docker
2e838b9
@supertoto29
changes to the code
b1c8136
@supertoto29
start to add error gestion
fd50b40
@supertoto29
add logs
2366e24
@supertoto29
fix bool function
7753fe7
@supertoto29
add verification of sensor version
87fafb0
@supertoto29
test for fixed branch
f99669a
supertoto29 and others added 19 commits October 25, 2021 20:42
@supertoto29
back to shared Queue
02feda4
@supertoto29
change serial constructor
d15e23b
@supertoto29
Merge pull request #8 from sonia-auv/fix/refactor-driver
bf83fad 
Fix/refactor driver
@supertoto29
fix workflow for package
b312ab4
@supertoto29
Merge pull request #9 from sonia-auv/fix/refactor-driver
f21bfa8 
fix workflow for package
@supertoto29
starting the custom packet. Problem with the array
403eace
@supertoto29
change to char array to test
514a824
@supertoto29
conversion from string to char array complete
e58ffe5
@supertoto29
prevent decay of array
3431ffc
@supertoto29
decay removed send and receive working
2e9dd97
@supertoto29
remove from local project definition
a0b2af2
@supertoto29
add send to ROS. to test
d28faa2
@supertoto29
test to change made to parse message
765b9ed
@supertoto29
fix export to ros hardcoded
47d2dec
@supertoto29
fix export to ros array to msg
c2d4130
@supertoto29
fix send message
098879d
@supertoto29
remove comments
96759dd
@supertoto29
remove sharedqueue not used
876e95f
@supertoto29
remove the custom packet from provider
6d09c59
@supertoto29 supertoto29 requested review from 0x72D0 and Kam0ss December 16, 2021 14:26
@supertoto29 supertoto29 self-assigned this Dec 16, 2021
0x72D0
0x72D0 approved these changes Dec 19, 2021
View reviewed changes
Copy link
Contributor

@0x72D0 0x72D0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@supertoto29 supertoto29 merged commit c13cc99 into master Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0x72D0 0x72D0 0x72D0 approved these changes

@Kam0ss Kam0ss Awaiting requested review from Kam0ss

Assignees

@supertoto29 supertoto29

Labels
Priority: High Type: Feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@supertoto29 @0x72D0