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

Hillshading #922

Closed
wants to merge 28 commits into from
Closed

Hillshading #922

wants to merge 28 commits into from

Conversation

usrusr
Copy link
Contributor

@usrusr usrusr commented Feb 11, 2017

Open todos:

  1. Alignment is sometimes off
  2. unfiltered upscaling is on Android
  3. the conversion from height values is just placeholder quality
  4. The decision between AwtLuminanceShadingComposite and AlphaComposite should either be properly exposed to client applications (or themes) or it should go (properly hardcoded, without pretensions of flexibility). Without a consistent implementation on the Android side I would suggest staying with alpha blending on both sides (I would personally prefer luminance blending, since I expect it to work better with existing themes)

…s in terms of image blending modes than AWT (colors can be mangled freely with a colormatrix, but blending will use the alpha from before the mangling)
… the wrong assumption that ways are pushed into PolyLineContainers more often than they are read)
…values corresponding to the mapping required by the Android implementation)
Conflicts:
	mapsforge-map-android/src/main/java/org/mapsforge/map/android/graphics/AndroidCanvas.java
	mapsforge-map-android/src/main/java/org/mapsforge/map/android/graphics/AndroidGraphicFactory.java
…t improves alignment and enables filtered upscaling on android,

unfinished renderscript alternative not removed yet (includes a nice workaround to enable rendersript kernels on rbg565)
@usrusr
Copy link
Contributor Author

usrusr commented Feb 12, 2017

alignment and unfiltered upscaling on android are fixed with latest commit (by switching from matrix-contolled scaling to src-rect/dest-rect controlled scaling), sorry for being a moving target

@devemux86
Copy link
Collaborator

devemux86 commented Feb 12, 2017

Thanks!

I'd like to parse the PR for some improvements like code formatting, methods order and rebase / squash in a new branch to be able to continue there.
Have you other immediate work to push or PR will be quiet for some time?

@devemux86 devemux86 mentioned this pull request Feb 12, 2017
@usrusr
Copy link
Contributor Author

usrusr commented Feb 12, 2017

I expect work on todo.3 (better hgt to shading) to be pretty much self-contained in a new implementation of o.mf.m.layer.hills.ShadingAlgorithm so it will smoothly merge with any changes you make to the stuff in the current PR. I'll just do it in a new branch so that I can keep the merging effort with your changes to the current PR on my side.

@devemux86
Copy link
Collaborator

devemux86 commented Feb 12, 2017

I'll merge this PR and any improvements in a new hillshading branch for users to start playing with it.
Then you can fetch it and post any new changes in that branch as new future PR, in order to update that.

BTW you can use GitHub markdown task lists in your first comment to indicate the completed tasks.

devemux86 pushed a commit that referenced this pull request Feb 12, 2017
@devemux86
Copy link
Collaborator

Rebased, squashed and merged in hillshading branch via f66021f, with minor improvements via 16be706.

Can you post any future PR work on that branch?

@devemux86 devemux86 added this to the 0.8.0 milestone Feb 26, 2017
devemux86 pushed a commit that referenced this pull request Feb 26, 2017
@devemux86 devemux86 changed the title Hillshading WIP Hillshading Feb 26, 2017
@devemux86
Copy link
Collaborator

Since regular Mapsforge API can continue as usual and for broader user visibility..
I rebased, squashed and merged via 7dd483c, with minor improvements via 1928ec9.

I included the hillshading rules in new renderTheme-v6 XML schema definition.
Also on Desktop Samples the 1st argument can be used as dem folder.

@devemux86 devemux86 closed this Feb 26, 2017
@usrusr usrusr deleted the hills branch October 6, 2017 00:52
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.

2 participants