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

Visual message when no data fetched in gyroscope #1785

Merged
merged 1 commit into from Jun 11, 2019

Conversation

ho-dor
Copy link
Contributor

@ho-dor ho-dor commented Jun 10, 2019

Fixes #1723

Changes: Graphs removed and a visual message to user when no data fetched

Screenshot/s for the changes:
Screenshot_20190610-231434_PSLab

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing:
gyro.zip

@ho-dor
Copy link
Contributor Author

ho-dor commented Jun 10, 2019

@CloudyPadmal @neel1998 please check.

recordedGyroArray = new ArrayList<>();
resetInstrumentData();
plotAllRecordedData();
}else{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Reformat

noSensorText.setAllCaps(true);
gyroLinearLayout.addView(noSensorText);


Copy link
Collaborator

Choose a reason for hiding this comment

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

reformat

@@ -133,6 +154,8 @@ public void onDestroyView() {
}
if (sensorManager != null) {
sensorManager.unregisterListener(gyroScopeSensorEventListener);
}else{
Copy link
Collaborator

Choose a reason for hiding this comment

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

reformat

@@ -831,4 +831,5 @@
<string name="setting_thermo_sensor_gain">setting_thermo_sensor_gain</string>
<string name="default_value_thermo">0</string>
<string name="thermo_default">°C</string>
<string name="no_data_recorded_gyro">No data recorded as the sensor is not present in the device. Try with an external sensor.</string>
Copy link
Collaborator

Choose a reason for hiding this comment

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

move to relevant section where variable related to gyro is

@ho-dor
Copy link
Contributor Author

ho-dor commented Jun 10, 2019

@CloudyPadmal done

@CloudyPadmal CloudyPadmal added the Status: Review Required Requested reviews from peers and maintainers label Jun 11, 2019
@CloudyPadmal CloudyPadmal merged commit 8ec739a into fossasia:development Jun 11, 2019
neel1998 pushed a commit to neel1998/pslab-android that referenced this pull request Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Review Required Requested reviews from peers and maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logged Data not showing graph in Gyroscope when sensor absent
3 participants