-
Notifications
You must be signed in to change notification settings - Fork 136
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
allow a selection on the exposure layer too #4786
base: develop
Are you sure you want to change the base?
Conversation
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.
LGTM
I have an issue with raster layer (not checking for selected features). |
hi @Gustry - I have little time for technical work now ;( |
No worries, I think I will target it for 5.0. I still need to polish this feature.
|
daedf0c
to
0b52df9
Compare
Raster exposure fixed |
0b52df9
to
c1f17f5
Compare
c1f17f5
to
8c7d305
Compare
@ismailsunni I updated this branch with develop to fix conflicts. But I have no idea what is wrong with Travis. I can't see any error. The make docker-test is fine on my local. Any idea? |
yeah, this is strange... I restarted Travis, let's see what happen. |
I already restarted many times ... |
09d5cc2
to
595ebf6
Compare
I will continue the PR, updating with develop |
595ebf6
to
6d959a7
Compare
This PR is leading to a crash of Travis. Something with signals from the map canvas. I was having trouble |
What does it fix?
I did that quickly. Maybe if we want it for 4.3, it needs some testing to check I didn't break anything or it's working correctly
it's a little bit more than one line like @timlinux said this morning ;-)
Checklist: