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

New functions to generate random latitude and longitude coordinates within a specified polygon #201

Merged
merged 15 commits into from
Nov 10, 2024

Conversation

efremropelato
Copy link
Contributor

Add two functions to generate random latitude and longitude coordinates within a specified polygon:

  1. NearbyGPSIntoPolygon: Generates a random point within the polygon, starting from a specified point.
  2. NearbyGPSIntoPolygonWhitoutStart: Generates a random point within the polygon without requiring a starting point.

Both functions ensure that the generated points lie within the boundaries of the loaded polygon with a GeoJson file.

@efremropelato
Copy link
Contributor Author

Test:

  1. GeoJson boundary polygon (blue):
Screenshot 2024-11-05 alle 23 07 32
  1. Tracking a random path (red) with NearbyGPSIntoPolygon function:
Screenshot 2024-11-05 alle 23 07 48

@hifly81 hifly81 requested review from hifly81 and ugol November 6, 2024 10:47
Makefile Show resolved Hide resolved
@efremropelato
Copy link
Contributor Author

Is something not working with this code?

I have more functions to add, but GitHub Flow keeps all commits in this PR, so creating a new one would just include these commits too. Do you want me to move these functions to the new JRv2 repo?

Thanks!

@vmarchese
Copy link
Collaborator

Ciao Efrem, dovresti lavorare sul tuo fork e ,quando pensi di aver finito una funzionalità completa, richiedere una PR.
Le PR devono avere uno scope limitato
Altrimenti ci arrivano tutte le tue push sul repository e non riusciamo a fare una seria review.

Per favore segui questa linea guida (o altre simili):
https://docs.github.com/en/enterprise-server@3.12/pull-requests/collaborating-with-pull-requests/getting-started/best-practices-for-pull-requests

@efremropelato
Copy link
Contributor Author

efremropelato commented Nov 9, 2024

Ciao, è quello che mi sembra aver fatto.
Senza un PR Template, non mi è chiaro cosa vi aspettate e se sto sbagliando qualcosa.

Questa PR include tutto il necessario per una singola funzione: visto che non avrebbe avuto senso creare una PR solo per caricare il geojson senza una funzione che ne usi le geometrie, giusto?
Altre funzioni (in altri branch nel mio fork, di cui non c'è ancora una PR) usano le stesse basi, come il caricamento del geojson:
continuo a creare branch separati per poter avere in futuro delle PR con scopi limitati,

Cosa intendi con "Altrimenti ci arrivano tutte le tue push sul repository e non riusciamo a fare una seria review."?
Non ho fatto altri push, oltre a quello iniziare e quelli richiesti da @hifly81.

@vmarchese
Copy link
Collaborator

io vedo 15 commit e 12 file modificati anche se non sono stati toccati (e.g.: stesse linee rimosse e aggiunte).
Comunque diamo un'occhiata e mergiamo la PR. La funzione è sicuramente utile

@efremropelato
Copy link
Contributor Author

Se disturba, perché non regolarizzare utilizzando un linting o formatting tool per golang. cosi il codice sarebbe omogeneo.

@vmarchese
Copy link
Collaborator

figurati, non è il disturbo. I linter ci sono ma in fase di build. In ogni caso, il tempo di rivedere le effettive modifiche e mergiamo

@vmarchese vmarchese merged commit a67c27f into jrnd-io:main Nov 10, 2024
2 checks passed
@efremropelato efremropelato deleted the feature/NearbyGPSIntoPolygon branch November 11, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants