-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
3D View upgrade and fixes: Visualizing the world imagery in 3D View #11214
3D View upgrade and fixes: Visualizing the world imagery in 3D View #11214
Conversation
0c38a40
to
1acbe89
Compare
@SebastianViollaz
Then you can open and build the project using Qt Creator. |
I find this: Here an overview of the mission: Null value in item It seems like the "L" is showed too but a null value too. No map loaded ROI not showed This is the flight plan that i'm using so you can test it |
@SebastianViollaz |
LaVervena.zip |
@SebastianViollaz |
Im testing with the SITL Ardupilot sim (master version) https://github.com/ArduPilot/ardupilot. (Arducopter) |
I think that in the mission that i provied i don use the ROI that is in the tool side bar in the plan editor. |
3ae5bd9
to
94dcfac
Compare
@SebastianViollaz |
Tested, properly working for me |
@SebastianViollaz |
5f3b3c5
to
8541e82
Compare
8541e82
to
55315c5
Compare
bf2e92c
to
e2b5117
Compare
e2b5117
to
18585d8
Compare
@@ -253,13 +253,20 @@ SettingsPage { | |||
text: qsTr("Select File") | |||
|
|||
onClicked: { | |||
var filename = qsTr(_viewer3DOsmFilePath.rawValue); |
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.
The qsTr here seems wrong. Unless I'm misunderstanding something qsTr should be used with string literals not variables.
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.
@DonLakeFlyer
Thanks for your comment. I just fixed that.
8726a7f
to
73f7313
Compare
@HTRamsey Can you look at the docs failure? I Quick Look by me seems to indicate that the check for a push has been removed and things which should be push only are running on pulls. |
…le is not visible when 3D View is open
…ctory, Viewer3D RTL bug fixed
…the Bing provider
73f7313
to
ba4af81
Compare
@DonLakeFlyer |
Great, thanks |
The following changes have been made to the 3D View:
An example of the new changes is illustrated in the figure below: