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

New Luos engine repo organization #174

Closed
wants to merge 96 commits into from
Closed

New Luos engine repo organization #174

wants to merge 96 commits into from
+1,417,890 −1,571

Conversation

nicolas-rabault
Copy link
Member

@nicolas-rabault nicolas-rabault commented Mar 23, 2022 •
edited by JeromeGalan
Loading

Comments for current PR

  • add your comments here
  • ...







!!! WARNING : DO NOT EDIT THE CHECKLIST BELOW !!!

Checklist : Merge to DEVELOP branch

Code Quality

  • Each function has a header (description, inputs, outputs)
  • Code is commented (particularly in hard to understand areas)
  • There are no new warnings that can be corrected
  • Commits policy is respected (constitancy commits, clear commits comments)


Tests Review

  • Manual Tests for new features have been reviewed
  • Automatic Tests for new features have been reviewed


Tests are passed the best as I can

  • Non regression existing tests are passed OK
  • New tests for new features & bug fixes are passed OK


Documentation

  • Documentation is up to date
  • Changelog for next release is up-to-date : fill here
  • If needed, tests are briefly described in test repository

nicolas-rabault and others added 30 commits March 23, 2022 11:00
@nicolas-rabault
Add Examples
d1aab55
@nicolas-rabault
Move Gate and Inspector to Tool_services
903c5a4
@nicolas-rabault
Move Robus into Network
aa5deb0
@nicolas-rabault
Move LuosHAL into Network/Robus/HAL
08ed2dc
@nicolas-rabault
Move Luos code into Luos_engine folder
5a5cb34
@nicolas-rabault
Change luos.h into luos_engine.h
8b05185
@nicolas-rabault
Create a luos.h file including luos_engine.h file for retrocompatibility
d3cc161
@nicolas-rabault
Rename luos_hal into robus_hal
883a3ed
@nicolas-rabault
Rename luos_hal into robus_hal in comments
1bcc268
@nicolas-rabault
Update project script for unit testing
998e439
@nicolas-rabault
Rename luos_hal into robus_hal in unit test
cf278c5
@nicolas-rabault
Update Unit Tests
a42a3a4
@nicolas-rabault
Split HAL into robus_HAL and Luos_HAL
41a1126
@nicolas-rabault
Check all examples on actions
9b2e7bd
@nicolas-rabault
Check all examples on actions
d95f6d5
@nicolas-rabault
Fix some examples compilations
ade7e74
@nicolas-rabault
Add Bootloader projects on Examples
f3a10a9
@nicolas-rabault
Update Readme
27dabee
@nicolas-rabault
Add a link checker on Readme
bfa2fe9
@nicolas-rabault
Manage Luos, Gate, and Inspector release GH actions
fdd128f
@nicolas-rabault
Rename Luos_engine folder into Engine
8857d38
@nicolas-rabault
Add a compilation message and separations indicating the selsected HA…
e3829a7 
…L and Network.
@nicolas-rabault
Add a bootloader compilation option for l0 boards
8eaa793
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-F072RB boards
d1ae482
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-F401RE boards
0e67935
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-G431KB boards
650a98a
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-G474RE boards
1ad1731
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-L432KC boards
b3ce4ee
@nicolas-rabault
Add a bootloader compilation option for NUCLEO-F410RB boards
b81fc68
@nicolas-rabault
Use the last FreeRTOS platformio lib instead of storing the entire code
58cbe35
nicolas cros and others added 24 commits April 7, 2022 18:29
update platformIO.ini file
451bc1b
Merge pull request #175 from Luos-io/feature/simpleFOC
795855b 
merge simpleFoc examples
@nicolas-rabault
Improve script following Platformio recommendations
f51f1b4
@nicolas-rabault
Fix a broken link on readme
a63cf3a
@BiliouriV
[Demo-motor demo app] Fix Routing tb filter function
8b310b6
@nicolas-rabault
Remove useless common environment on bootloaders
35e6f70
Merge pull request #182 from Luos-io/feat/pio_feedbacks
57dd93e 
Integrate Platformio feedback about this repository
fix PTP handling during detection
c19d884
@nkrs-lab
Merge pull request #185 from Luos-io/fix/detection_arduino
32f7f8f 
fix PTP handling during detection
@nicolas-rabault
Avoid a warning on Luos_assert(__FILE__, __LINE__) by explicitly cast…
4ae179e 
…ing file into char*
@nicolas-rabault
OD_time manage time on a double instead of float
b190e56
@nicolas-rabault
OD_time manage ns time values conversion.
fd9630c
@nicolas-rabault
Add string dependencies on OD
98db539
@nicolas-rabault
Remove token management from timestamp
a501831
@nicolas-rabault
Improve some comments
826e52d
@nicolas-rabault
Update examples to deal with the new timestamp API
7a3e53a
@nicolas-rabault
Update and improve Timestamp unit tests
f3dc446
@nicolas-rabault
Update timestamp comments.
7ad6452
@BiliouriV
[Timestamp] Fix timestamp bugs -> size + ack send
8a8d342
Update timestamp unit test
f239a9e
Merge pull request #183 from Luos-io/feat/timestamp
cac0598 
Feat/timestamp
Clean sources path
6163a4a
Update versions for release 2.4.0
e8fc008
Merge pull request #181 from Luos-io/version
6eeb181 
Update versions for release 2.4.0
JeromeGalan
JeromeGalan approved these changes Apr 25, 2022
View reviewed changes
Copy link
Member

@JeromeGalan JeromeGalan left a comment

Choose a reason for hiding this comment

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

Good job to all devs !

@nicolas-rabault
Copy link
Member Author

Main is now the root branch of this repo. We don't merge this PR because of it.

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

@JeromeGalan JeromeGalan JeromeGalan approved these changes

@Salem-Tho Salem-Tho Awaiting requested review from Salem-Tho

Assignees

@nicolas-rabault nicolas-rabault

Labels
None yet
Projects
Luos contribution project
Status: Todo
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rename master to main.
5 participants
@nicolas-rabault @JeromeGalan @K0rdan @BiliouriV @nkrs-lab