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

Added Visual Studio Code to pkg installer #11

Merged
merged 5 commits into from
Feb 8, 2019
Merged

Conversation

Rafcin
Copy link
Contributor

@Rafcin Rafcin commented Feb 8, 2019

I added a Visual Studio Code to the pkg installer, its not clean but it should install it correctly on all the Jetson devices and its a much better editor than Gedit or other arm based editors.

@rbonghi
Copy link
Owner

rbonghi commented Feb 8, 2019

Thank you @Rafcin I'm checking!

@Rafcin
Copy link
Contributor Author

Rafcin commented Feb 8, 2019

Awesome! The gist files if they correctly install will replace the ones in VS and compile with Electron, if they do then all is good 👍. Also VSCode hopefully will be useful to people because of it's customization features + workspace and git control.

@rbonghi rbonghi merged commit 3023c96 into rbonghi:master Feb 8, 2019
@rbonghi
Copy link
Owner

rbonghi commented Feb 8, 2019

I have only fixed the size of the window and add an extra comment on the patch menu about the wifi fix.

@Rafcin
Copy link
Contributor Author

Rafcin commented Feb 8, 2019

Awesome!

@Rafcin
Copy link
Contributor Author

Rafcin commented Feb 8, 2019

I'm going to add some more stuff over the weekend regarding ROS Melodic, there's a lot of bugs I've found over time that probably should be looked at. Also have you considered switching the catkin_make to Catkin build?

@Rafcin
Copy link
Contributor Author

Rafcin commented Feb 8, 2019

Also one thing that I think will be useful is adding RTABMAP and having it compile from scratch becuase it allows you to also use the Zed SDK vs when you install it through apt it doesn't have that enabled/pathway set.

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