This repository has been archived by the owner on Apr 12, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2
Ti comm dev #17
Draft
ihagedo
wants to merge
60
commits into
master
Choose a base branch
from
ti_comm_dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Ti comm dev #17
Changes from 24 commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
2a64324
modified sizes of objects
ihagedo cb14716
Modify makefile and SERIAL_TEST to try to find issue with makefile.
zghera 301dc54
bag test
ihagedo 7c0a32e
deleted outdated file
ihagedo a210a2b
added bagfile test
ihagedo 15a8da1
Merge branch 'isaac_dev' of https://github.com/Autonomous-Motorsports…
ihagedo d10c1ff
added bagfile test data
ihagedo d09125d
Merge branch 'isaac_dev' of https://github.com/Autonomous-Motorsports…
ihagedo 60bcc09
serial comm
ihagedo 158bb48
Merge branch 'ti_comm_dev' of https://github.com/Autonomous-Motorspor…
7bd98b1
Serial Communiction files updated. ECU_TEST works well with debug opt…
6a3331c
Update .gitattributes
ihagedo 379d173
cleaning lfs
ihagedo 5397dc0
added libserialport submodule
ihagedo 4055ea4
added readme
ihagedo 6fd1038
Update README.md
ihagedo 86a0c0e
Merge pull request #14 from Autonomous-Motorsports-Purdue/master
ihagedo 8450f33
fixed structural issues for packet
ihagedo 03b1c91
Merge branch 'isaac_dev' into ti_comm_dev
ihagedo d802305
Update amp_serial_jetson_ECU_TEST.cpp
ihagedo 8add71c
Delete amp_serial_jetson_ECU_TEST.o
ihagedo c93c273
Delete amp_serial_jetson
ihagedo 9ed89b1
Delete amp_serial_jetson_SERIAL_TEST.o
ihagedo 3cde596
Update amp_serial_jetson.h
ihagedo e7c275e
added number parameters
525baf5
seperated library files
ihagedo de488c6
integrated new library into build
ihagedo 454b19c
Merge branch 'ti_comm_dev' of https://github.com/Autonomous-Motorspor…
a79bb43
ecu test working properly
ihagedo 2b55e73
Gitignore vim swap files
bitmaks 5df3e51
Add read command from file functionality and delay flag to debugger
bitmaks 2fb37c6
Improve help description and add exit after help menu
bitmaks dc38afc
Simplify speed and angle read function
bitmaks ceee7af
Refactor to be more readable
bitmaks 0575b21
Merge pull request #25 from aaaakshat/ti_comm_dev
bitmaks 9014549
Merge branch 'ti_comm_dev' of https://github.com/Autonomous-Motorspor…
86efbcb
restructured ti_comm dir
c58950b
Merge branch 'master' of https://github.com/Autonomous-Motorsports-Pu…
27c88aa
Added includes for SERIAL_TEST
73632e0
segrated old and new lib, resolved new lib build issues
0ef4c58
restructuring ti_comm/src
ihagedo 39092fd
Add libserialport patch for linux configs
bitmaks 06b75c8
Fix patch whitespacing issue
bitmaks bf52685
Add build script
bitmaks bd3067a
Modified build.sh to create libserialport obj files.
zghera ae501bf
Recognize if patch has been applied in build.sh
bitmaks 0421949
FIx scriptDir error on linux
bitmaks 6f3e346
Fix typo
bitmaks 07232ea
Change shell to bash so source works.
zghera 6d8fbc7
Don't fetch libserialport if patch has been applied
bitmaks 0f613ca
Gitignore MacOS trash and debug files
bitmaks f87eca5
fixed ECU
ihagedo a2216e6
Merge branch 'ti_comm_dev' of github.com:Autonomous-Motorsports-Purdu…
ihagedo adeee69
added micro sleep
ihagedo 9704f02
Fix serial pkt and control pkt memcpy issue
bitmaks a01c81d
Make control pkt use unsigned not signed char
bitmaks 7bd6bfe
Add debug messages to serial and ecu test
bitmaks 0368eb4
serial test work
bitmaks ec65cf8
Update amp_serial_jetson (please work)
bitmaks 0bc09d4
Debugging 0 serial packets. ti_comm node is crapping out now.
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[submodule "src/ti_comm/src/libserialport"] | ||
path = src/ti_comm/src/libserialport | ||
url = https://github.com/sigrokproject/libserialport.git |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -11,8 +11,14 @@ rosdep install --from-paths src --ignore-src -r -y | |||||||||||||||||||||||||||||||||||
To learn more about [rosdep](http://wiki.ros.org/rosdep), check out the [ROS wiki](http://wiki.ros.org/rosdep). | ||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||
## Building | ||||||||||||||||||||||||||||||||||||
#### ROS | ||||||||||||||||||||||||||||||||||||
As mentioned in the [ROS core tutorials on building packages](http://wiki.ros.org/ROS/Tutorials/BuildingPackages), run the following two commands at the top of the directory (in AMP-v1) to build each of the ROS packages in our project. | ||||||||||||||||||||||||||||||||||||
``` | ||||||||||||||||||||||||||||||||||||
catkin_make | ||||||||||||||||||||||||||||||||||||
source devel/setup.bash | ||||||||||||||||||||||||||||||||||||
``` | ||||||||||||||||||||||||||||||||||||
#### LIBSERIALPORT | ||||||||||||||||||||||||||||||||||||
At the top of each branch your want to build the library for you should run the following command and then refer to the readme in src/ti_comm/src | ||||||||||||||||||||||||||||||||||||
``` | ||||||||||||||||||||||||||||||||||||
git submodule update --init --recursive | ||||||||||||||||||||||||||||||||||||
``` | ||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,22 @@ | ||
# Compiler Option | ||
CC = g++ | ||
WARNING = -Wall -Wshadow --pedantic | ||
ERROR = -Wvla -Werror | ||
GCC = g++ -std=c++11 -lstdc++ -lm | ||
PKG_CONFIG = pkg-config | ||
CFLAGS = -g $(WARNING) $(ERROR) $(shell $(PKG_CONFIG) --cflags libserialport) | ||
LIBS = $(shell $(PKG_CONFIG) --libs libserialport) | ||
|
||
# Compiler Flags | ||
CFLAGS = -g -Wall | ||
TESTFLAGS = -DDEBUG=1 -DDEBUG_TX=1 -DDEBUG_RX=1 | ||
SOURCES = $(wildcard *.cpp) | ||
|
||
default: all | ||
BINARIES = $(SOURCES:.cpp=) | ||
|
||
all: amp_serial_jetson.o libserialport-0.1.0/serialport.o libserialport-0.1.0/linux_termios.o | ||
$(CC) $(CFLAGS) -o amp_serial_jetson amp_serial_jetson.o libserialport-0.1.0/serialport.o libserialport-0.1.0/linux_termios.o -ludev | ||
%: %.cpp | ||
$(GCC) $(CFLAGS) $< $(LIBS) -o $@ | ||
|
||
amp_serial_jetson.o: amp_serial_jetson.cpp amp_serial_jetson.h util.h amp_err.h | ||
$(CC) $(CFLAGS) -c amp_serial_jetson.cpp | ||
debug_%: %.cpp | ||
$(GCC) $(CFLAGS) $(TESTFLAGS) $< $(LIBS) -o $@ | ||
|
||
sample_publisher: sample_publisher.o | ||
$(CC) $(CFLAGS) -o sample_publisher sample_publisher.o | ||
all: $(BINARIES) | ||
|
||
sample_publisher.o: sample_publisher.cpp | ||
$(CC) $(CFLAGS) -c sample_publisher.cpp | ||
clean: | ||
rm $(BINARIES) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
How to Build Libserialport | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove this file as a whole if you agree with the suggestion to move its contents t the top-level README |
||
1. Add Submodule From Root | ||
* git submodule update --recursive | ||
2. Build Library | ||
* ./autogen.sh | ||
* ./configure | ||
* make | ||
* sudo make install | ||
3. Add Library to Path | ||
* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the purpose of this empty file?