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

[issue #702 #719] Change in uav_viewer_py#722

Merged
aitormf merged 2 commits into
JdeRobot:masterfrom
RoboticsLabURJC:master
Apr 5, 2017
Merged

[issue #702 #719] Change in uav_viewer_py#722
aitormf merged 2 commits into
JdeRobot:masterfrom
RoboticsLabURJC:master

Conversation

@Diegojnb
Copy link
Copy Markdown
Contributor

No description provided.

@aitormf
Copy link
Copy Markdown
Collaborator

aitormf commented Mar 29, 2017

Hi,
I'm trying the app and all work fine. The only things I've seen that don't fit are the play button and color filter checkbox.
captura de pantalla de 2017-03-29 17-16-23
When you delete them from GUI, this PR can be merged
Good job

@Diegojnb
Copy link
Copy Markdown
Contributor Author

Diegojnb commented Apr 2, 2017

Hi,
I have copied the model that is currently in the official repository under the name of uav_viewer_py and I have changed the way that the speed commands with two joysticks are modified, as would be piloted with a command of any dron.

If it is mandatory to make this change I modify it but currently those buttons appear in the official repository.

@aitormf
Copy link
Copy Markdown
Collaborator

aitormf commented Apr 4, 2017

This is the actual GUI of Uav Viewer py:
captura de pantalla de 2017-04-04 09-38-30
How you can see, they aren't in the GUI. Delete them please

@Diegojnb
Copy link
Copy Markdown
Contributor Author

Diegojnb commented Apr 4, 2017

This is actually look like

uav_viewer_py

@aitormf aitormf merged commit baf3fc7 into JdeRobot:master Apr 5, 2017
@aitormf
Copy link
Copy Markdown
Collaborator

aitormf commented Apr 5, 2017

Merged,

Thanks @Diegojnb

@shivangg
Copy link
Copy Markdown
Contributor

Please update the ui_gui.ui to current user interface. It still displays this UI when the ui_gui.ui file is opened in Qt Creator

image

rather than the current UI
image

Only after this update, new features to the GUI of uav_viewer_py can be added like #660

@jmplaza
Copy link
Copy Markdown
Member

jmplaza commented Apr 27, 2017

Solved at #790 @shivangg

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.

4 participants