Move SQL Query Out of serialReceiver.py Script

[PeiyuanQi]

Leverage SQL Query code from serialReceiver.py into several functions in a new python file named sqlDriver.py, then serialReceiver.py should import related functions from sqlDriver.py and avoids duplicated code related to sql query in serialReceiver.py.

• Database configuration (port, host, username, password) should be set in sqlDriver.py, when serialReceiver.py call functions in sqlDriver.py, it only passes parameters related to each table (database name, table name, query column head, etc.)

[adamvnovak]

I am very interested in working on this task.
Adam

[PeiyuanQi]

I am very interested in working on this task.
Adam

Great Adam, you could first go through the file and understand how the connection is built and what are our query SQL sentences. Then you could try to abstract the code from serialReceiver and warp then with a class. The ultimate goal is create a class, providing some functions to do the same thing to avoid redundant code.

To start
Read the file. Figure out the interfaces to be implemented. Ask questions anytime when you read the file.

[adamvnovak]

Thank you for the info!! I have 3 exams next week that I am preparing for right now but once those are over I will begin to look heavily into the Solar Car SQL code. Adam

…

On Fri, Mar 22, 2019 at 8:15 AM Peiyuan Qi ***@***.***> wrote: I am very interested in working on this task. Adam Great Adam, you could first go through the file and understand how the connection is built and what are our query SQL sentences. Then you could try to abstract the code from serialReceiver and warp then with a class. The ultimate goal is create a class, providing some functions to do the same thing to avoid redundant code. To start Read the file. Figure out the interfaces to be implemented. Ask questions anytime when you read the file. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_USCSolarCarStrategyTeam_plotly-2Ddashboard_issues_4-23issuecomment-2D475660108&d=DwMFaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=DkLu-G6JMRrETfPYQby5lg&m=g6PhYrN0opqnkcAXyEQB_n2PCJnTdLGDtJhXaBvPalU&s=v-u8L46a1i7hD1hizj5QaRCGuaADj4oHU1jpxrFPO2c&e=>, or mute the thread <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AoqINCoz9fl8np-5F1Vkf3QN1rZYgFwS5Iks5vZPOXgaJpZM4bZbVx&d=DwMFaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=DkLu-G6JMRrETfPYQby5lg&m=g6PhYrN0opqnkcAXyEQB_n2PCJnTdLGDtJhXaBvPalU&s=v2xToZOnHitkUwzdqy3sQHtzot6aV3sEkxNCwVXyH_U&e=> .

[PeiyuanQi]

This issue is still open to solve