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
feat: recording functionality in compass instrument #1555
Conversation
compassLogger.writeCSVFile("Timestamp,DateTime,Bx,By,Bz"); | ||
recordData(); | ||
writeHeaderToFile = !writeHeaderToFile; | ||
} else{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Smit2010 Please reformat the code, it shouldn't be this way.
65913b0
to
0d809e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @Smit2010 👍
} | ||
|
||
public String getBx() { | ||
return Bx; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you append this.
to these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you suggesting to append this. to getters also or only in setters? @CloudyPadmal
public CompassData() { | ||
} | ||
|
||
public CompassData(String x, String y, String z) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use standard practice in object creation. We use the same name and use this
key word to refer to the class instance. Change x y z to Bx By and Bz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated. @CloudyPadmal
b81a5ea
to
417435d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @Smit2010 👍
Fixes #1371 Recording functionality added in compass instrument
Changes: added function of record data in CompassActivity and created a new class of compassData
Screenshot/s for the changes:
Checklist: [Please tick following check boxes with
[x]
if the respective task is completed]strings.xml
,dimens.xml
andcolors.xml
without hard-coding themstrings.xml
,dimens.xml
orcolors.xml
APK for testing:app-debug.zip