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

allow a selection on the exposure layer too #4786

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Gustry
Copy link
Contributor

@Gustry Gustry commented Dec 14, 2017

What does it fix?

  • Ticket: #
  • Funded by: DFAT
  • Description: Request from @Charlotte-Morgan during the zoom walkthrough

selection

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:

  • Commit messages are descriptive and explain the rationale for changes
  • Add to the changelog in metadata.txt if it's a new feature
  • Unit test for new code added
  • Request someone to review or test your PR

Copy link
Member

@ismailsunni ismailsunni left a comment

Choose a reason for hiding this comment

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

LGTM

@Gustry
Copy link
Contributor Author

Gustry commented Dec 20, 2017

I have an issue with raster layer (not checking for selected features).
I'm targetting this for 5.0 as we would like to release 4.3 asap now

@Charlotte-Morgan
Copy link
Member

hi @Gustry - I have little time for technical work now ;(

@Gustry
Copy link
Contributor Author

Gustry commented Dec 21, 2017

No worries, I think I will target it for 5.0. I still need to polish this feature.

  • Note for myself, the green bounding is not correct as you may have a long road going out of the selection area. But I don't think I can do something for this one. It works better for buildings or point exposure.

@Gustry
Copy link
Contributor Author

Gustry commented Jan 9, 2018

Raster exposure fixed

@Gustry
Copy link
Contributor Author

Gustry commented Mar 12, 2018

@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?

@ismailsunni
Copy link
Member

yeah, this is strange... I restarted Travis, let's see what happen.

@Gustry
Copy link
Contributor Author

Gustry commented Mar 13, 2018

I already restarted many times ...

@Gustry Gustry force-pushed the selected_exposure branch 3 times, most recently from 09d5cc2 to 595ebf6 Compare March 13, 2018 14:34
@ismailsunni
Copy link
Member

I will continue the PR, updating with develop

@Gustry
Copy link
Contributor Author

Gustry commented Apr 10, 2018

This PR is leading to a crash of Travis. Something with signals from the map canvas. I was having trouble

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants