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

Create pylint.yml #19

Merged
merged 36 commits into from
Nov 30, 2023
Merged

Create pylint.yml #19

merged 36 commits into from
Nov 30, 2023

Conversation

christianvluu
Copy link
Collaborator

@christianvluu christianvluu commented Sep 20, 2023

Pylint to check easy linting errors on newly edited files. This will ignore old files that are not edited.

@christianvluu christianvluu marked this pull request as ready for review November 19, 2023 06:18
@christianvluu christianvluu marked this pull request as draft November 19, 2023 06:23
@christianvluu
Copy link
Collaborator Author

See last two commits for failure/pass example.

@christianvluu christianvluu marked this pull request as ready for review November 19, 2023 07:16
@christianvluu christianvluu merged commit 4826444 into main Nov 30, 2023
3 checks passed
@christianvluu christianvluu deleted the actions branch November 30, 2023 03:40
Jackack pushed a commit that referenced this pull request Feb 3, 2024
Jackack pushed a commit that referenced this pull request Feb 3, 2024
Jackack added a commit that referenced this pull request Feb 14, 2024
* updated path planner and autonsystem, added option for building trajectory from __init__  parameter of trajectory.py

* [BETA] obstacle avoidance

* added halfplane constraints to MPC

* Switched path planning to use offsets along the path normal

* Cleaned up unused code, added comments

* CI (#19)

* pylint fixes

more pylint fixes

pylint fixes for 2d sim and other existing modules

pylint fixes yet again

pylint fixes yet again #2

* removed ROSbags

* added changes requested

* Fixed merge issues
Added exec permission to velocity_ui.py

* Updated velocity ui window title
added if statement for selecting velocity source in launch config

---------

Co-authored-by: Christian Luu <[email protected]>
Jackack added a commit that referenced this pull request Mar 13, 2024
…eserved (#47)

* updated path planner and autonsystem, added option for building trajectory from __init__  parameter of trajectory.py

* [BETA] obstacle avoidance

* added halfplane constraints to MPC

* Switched path planning to use offsets along the path normal

* Cleaned up unused code, added comments

* CI (#19)

* pylint fixes

more pylint fixes

pylint fixes for 2d sim and other existing modules

pylint fixes yet again

pylint fixes yet again #2

* removed ROSbags

* added changes requested

* Updated main.launch

Co-authored-by: Mehul Goel <[email protected]>
Co-authored-by: TiaSinghania <[email protected]>
Co-authored-by: PatXue <[email protected]>

* fixed main.launch typo

* Fixed topic remapping

* fixed topic remp, added dummy subscriber for debug

* Fixed heading gain, added ghost NAND feature

* removed dummy node (was used for debug)

* Commented out MPC halfplane constraints,
Updated sim launch file

* Added exception for path run out for MPC, stanley

Added more mock roll paths and starting poses

expanded MPC traj index search window

* Added debug steer routine

* updated the cut path

* changed controller type

* Added debug topic for filter  <-> gnss distance

* Turned on debug logging for INS

* set debug to true in INS_params.yml

* removed duplicate publish_rtk_err.py
fixed typo in publish_rtk_err.py variable naming

* Fixed pylint warnings

* removed duplicate publish_rtk_err.py

* removed unusued vars in MPC

---------

Co-authored-by: Christian Luu <[email protected]>
Co-authored-by: Mehul Goel <[email protected]>
Co-authored-by: TiaSinghania <[email protected]>
Co-authored-by: PatXue <[email protected]>
Co-authored-by: Buggy <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants