Skip to content
This repository was archived by the owner on Feb 5, 2024. It is now read-only.

Conversation

@kgrewal26
Copy link
Contributor

Description

Wrote up a quick header file to interface with the VN300 - mainly just GPS driver for now.

Asana Task: https://app.asana.com/0/187002107997407/1201573609333245/f

@kgrewal26 kgrewal26 self-assigned this Feb 25, 2022
public:

//added to make sure people dont accidentally initialize
VN300_GPS(const VN300_GPS*) = delete;
Copy link
Member

Choose a reason for hiding this comment

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

wait what does this do

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not gonna lie I don't fully remember.

I copied this over from a previous singleton I wrote a while back - If I cant remember the reason for it I'll delete it

void setGPSData();

//Returns GPS data struct
VN300_GPSData getGPSData();
Copy link
Member

Choose a reason for hiding this comment

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

can we get a bit more info on the gps data struct

Copy link
Member

Choose a reason for hiding this comment

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

nvm it's above I found it i'm stupid

//Private constructor to prevent multiple instances
VN300_GPS();

int registerID;
Copy link
Member

Choose a reason for hiding this comment

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

what would registerID do

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Honestly not too sure yet - I just left it there incase its eventually needed.

Will remove it if its not.

@antholuo
Copy link
Member

Looks good to me? Seems logical and functional.

@kgrewal26
Copy link
Contributor Author

kgrewal26 commented Mar 28, 2022

This wont build yet because of the exceptions issue. If you run it with "-fexceptions" appended at the end of the the build command it'll work tho.

I will go back and handle the exceptions once i know everything works.

Also, not all the VN library files are needed. I can go in and delete these once I figure out which ones we need and which ones we dont.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants