locationtech / proj4j

Java port of the Proj.4 library for coordinate reprojection
Other
184 stars 73 forks source link

Add geo projection in Registry #26

Closed Yaqiang closed 5 years ago

Yaqiang commented 5 years ago

Add geo projection in Registry, so Proj4Parser can get GeostationarySatelliteProjection with the code: projection = registry.getProjection(s);

pomadchin commented 5 years ago

Hey @Yaqiang can you create a cleaner PR? Just a new one with a single singed commit.

pomadchin commented 5 years ago

hey @Yaqiang, if it's a fresh commit it is enough just to do git commit -s -m "commit message" actually flag -s adds a footer that is required.

pomadchin commented 5 years ago

Can you create a new PR that has a single clean commit?

Yaqiang commented 5 years ago

I tried your recommond:

D:\MyProgram\java\proj4j>git commit -s -m "add geo projection in registry" On branch master Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean

It seems nothing happen. Maybe you can close this pull requests, and revise the code yourself. The purpose is problem solving. Thanks!

pomadchin commented 5 years ago

Hey @Yaqiang try the following:

# save your current changes in case you loose everything during your next steps
# git supports branches
git checkout -b tmp
# lets add a remote (the current) repo as an upstream repo
git remote add upstream git@github.com:locationtech/proj4j.git
# fetch it
git fetch upstream
# rebase master on top of it
git checkout -f master && git reset --hard f900cff && git rebase upstream/master && git push origin -f master
# create a new feature branch with the stuff you want to contribute
git checkout -b feature/add-into-registry
# add your code changes and after that do 
git add . && git commit -s -m "add geo projection in registry" && git push origin feature/add-into-registry

After that try to create a new PR.

pomadchin commented 5 years ago

I guess it should be a commit with the only change introduced into Registry.java:

register("geos", GeostationarySatelliteProjection.class, "Geostationary Satellite");

If that's what it is (since it's hard to figure out what had been changes, since you've rebased differently), I can create a PR myself.

pomadchin commented 5 years ago

@Yaqiang check it out https://github.com/locationtech/proj4j/pull/27

Yaqiang commented 5 years ago

@pomadchin You are right with the only change line code. So I will close this pull request.