Skip to content

Conversation

@neel1998
Copy link
Member

Fixes #1719

Changes: Created a layout to control servo

Screenshot/s for the changes:

robotic_arm_psl

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:
robotic_arm.zip

@CloudyPadmal @mariobehling @adityastic kindly have a look at this. I thought it would be much easier to control 4 servo in landscape mode , that's why implemented this layout. As @mariobehling suggested i have added circular view to control angle of servo

@mariobehling
Copy link
Member

This is illogical. Please refer to the picture below. Some servos can move 180 degress, some 360 degrees. We can go into detail here later. But, please implement circles.

circle-41073_960_720

@mariobehling
Copy link
Member

46331197-angle-45-360-degrees-circle-icons-geometry-math-signs-symbols-full-complete-rotation-arrow-flat-icon

@neel1998
Copy link
Member Author

@mariobehling i was thinking we had servo that rotates 0-180 degress that's why i made this design . I will change it and make a new layout today :)

@neel1998
Copy link
Member Author

@mariobehling @CloudyPadmal @adityastic kindly have a look at this new layout
Screenshot_20190518-181620

@neel1998
Copy link
Member Author

@mariobehling @CloudyPadmal kindly have a look the new layout in previous comments. Thanks

@mariobehling
Copy link
Member

Is it possible to enter the numbers manually as well by tapping on the number field? Please ensure this is working too.

@neel1998
Copy link
Member Author

@mariobehling right now I haven't added that. I was waiting for you to confirm the basic layout, after that i will add additional features

@mariobehling
Copy link
Member

Thanks. I think the basic layout is good. I see that you introduce a new color in the app. I don't remember that any other instrument uses blue. But, we can work on details later. Please implement the direct number entering functionality.

After that please continue working on the instrument. Details are here: #1734

@cweitat
Copy link
Contributor

cweitat commented May 22, 2019

You can refer to the knobs used at Power Source instrument, also please note codacy

@neel1998
Copy link
Member Author

@mariobehling @CloudyPadmal @cweitat I have created a new Layout for the servo controller. I have made the layout in a way that user won't need to scroll make it convenient to control the servos . Also i have made the parameters of the layouts dynamic so it would adjust based on device screen size. The drag and drop of the servo is working fine as seen in gif. I will make the timeline more precise in another PR. kindly have a look

20190522_200026
(separation in first 2 timeline is not properly visible in gif i don't know why, kindly refer to the screenshot)

robotic_arm_new.zip

@neel1998
Copy link
Member Author

@mariobehling since i have changed the whole layout, i think it would be better not to split this PR. I will make separate PRs for now onward changes

@neel1998
Copy link
Member Author

@mariobehling regarding the editable degree values, i am still working on it, because keyboard in the landscape mode is not responding very well.But i will figure out it really soon.

@adityastic
Copy link
Contributor

@neel1998 looks great

@mariobehling
Copy link
Member

Thank you. Let's get other parts done in follow up PRs.

@mariobehling mariobehling merged commit cf92bc1 into fossasia:development May 24, 2019
neel1998 added a commit to neel1998/pslab-android that referenced this pull request Jul 30, 2019
…sasia#1732)

* layout to control servo of robotic arm ceated

* new layout created with circular seekbars

* started working on drag and drop feature for robotic arm

* drag and drop of servo controllers working, timeline and UI needs to be changed

* timeline layout created, timeline controlls panel added
@neel1998 neel1998 deleted the robotic_arm branch August 27, 2019 13:59
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.

Servo motor control option

4 participants