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

Feature/can parser #5

Open
wants to merge 36 commits into
base: development
Choose a base branch
from
Open

Feature/can parser #5

wants to merge 36 commits into from

Conversation

Laszlo-Roovers
Copy link

Proposed changes

Dockerization

Types of changes

What types of changes does your code introduce to this repository?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • [ x] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

MathijsMoonen and others added 30 commits June 14, 2022 11:05
@MathijsMoonen
Create .gitignore
7c4590f
@MathijsMoonen
Moved CAN Parser stuff to StellaCANRead Proxy from Stratviz Proxy
76e6b91
@MathijsMoonen
Fixed paths for messages and typedefs.csv to enable testing
ea6ec98
@EmreAydo
Implemented printUniqueMessageTypes function
8141043
@EmreAydo
Implemented parseOverview function
850b5e6
@MathijsMoonen
Added unit tests for parseTimestamp, parseID and parseDataString. Als…
147a91e 
…o resolved all errors caused by these tests.
@MathijsMoonen
Merge pull request #1 from STE-Telemantics/feature/parseOverview
ed9be9b 
Implemented parseOverview function
@MathijsMoonen
Merge pull request #2 from STE-Telemantics/feature/printUniqueMessage…
7f1df81 
…Types

Implemented printUniqueMessageTypes function
@MathijsMoonen
divided parseTimestamp into two separate functions that perform the s…
79002e4 
…ame task
@MathijsMoonen
Merge branch 'feature/CANParser' of https://github.com/STE-Telemantic…
22d14c7 
…s/StellaCANRead-Proxy into feature/CANParser
@EmreAydo
First basic implementation for test cases of CANParser, MessageObject…
b09f98d 
… and TypeDefObject
@EmreAydo
Added missing endianness variable to the output of parseOverview
21f150a
@MathijsMoonen
Merge pull request #3 from STE-Telemantics/bugfix/missing_endianness
7412752 
Added missing endianness variable to the output of parseOverview
@MathijsMoonen
removed unused method, made some methods static
7bb5b5a
@MathijsMoonen
Merge branch 'feature/CANParser' of https://github.com/STE-Telemantic…
70ff78f 
…s/StellaCANRead-Proxy into feature/CANParser
@MathijsMoonen
added some imports and made a method static
c5d9335
@EmreAydo
Added additional tests to CANParserTest.java
f867bd7
@EmreAydo
Added additional tests to CANParserTest.java
09d7759 
Fixed some leading whitespaces in String arrays of MessageObject.java
@MathijsMoonen
Merge branch 'feature/CANParser' into feature/test_emreparser
24cfea4
@MathijsMoonen
Merge pull request #4 from STE-Telemantics/feature/test_emreparser
301dd32 
Feature/test emreparser
@MathijsMoonen
updated determineBits() to account for 'uint8_t:2' and other similar …
12823bd 
…data types. Also added numerous test cases for these types.
@MathijsMoonen
Merge branch 'feature/CANParser' of https://github.com/STE-Telemantic…
3bc30d6 
…s/StellaCANRead-Proxy into feature/CANParser
@MathijsMoonen
Added final determineBits() test cases. Finished determineBits() impl…
381115b 
…ementation.
@MathijsMoonen
Added testcases for determineConcreteData(), also fixed all issues th…
424d974 
…at were found using the tests
@MathijsMoonen
Added final tests, resolved their issues, and implemented a function …
6be061c 
…for the friday demo to show that it works
@DanielVanRijswijk
Updated CANEnumParser to use Objects rather than String
01db471
@MathijsMoonen
Adjusted testcases so that they work with the previously committed ch…
f707c35 
…anges
@MathijsMoonen
Update .gitignore
246a2e7
@DanielVanRijswijk
Updated files for demo friday
d9a8a1d
@DanielVanRijswijk
Create java.config
92bd878
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Laszlo-Roovers @MathijsMoonen @EmreAydo @DanielVanRijswijk @youssef-selim