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

Rohversion fertig, bitte Reviewen! #1

Open
yesdavid opened this issue Apr 26, 2019 · 2 comments
Open

Rohversion fertig, bitte Reviewen! #1

yesdavid opened this issue Apr 26, 2019 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@yesdavid
Copy link
Member

yesdavid commented Apr 26, 2019

Liebe ISAAKs,
ich habe @whamer s und meinen Code kombiniert und es funktioniert.
Könnte sich bitte jemand noch die beiden Probleme mit dem Finden der Maxima bei der NN-Distanz und maximalen Flächenzunahme anschauen?
David

https://github.com/ISAAKiel/lecAAR/blob/master/R/lecAAR-Sample.R

@yesdavid yesdavid added the help wanted Extra attention is needed label Apr 26, 2019
@nevrome
Copy link
Member

nevrome commented May 3, 2019

Was genau stellst du dir unter diesem Review vor, @yesdavid?

  • Lösung der genannten Probleme? Sind die im ISAAK-Treffen-Protokoll erklärt?
  • Test des vorhandenen Codes auf Funktionalität?
  • Umbau der Repositorystruktur zu einem R-Paket?
  • Verbesserung von Algorithmen und Implementierung?
  • Verbesserung des Code Stils?

Ich möchte dich nicht hängen lassen, weiß aber nicht so richtig, ob ich überhaupt gefragt bin, oder ob das ein Aufruf an @whamer und andere ist, die am letzten Treffen teilgenommen haben.

Ich schlage vor, Kommentare im Code und die Diskussion in diesen Issues hier auf Englisch zu führen. Das erlaubt es der internationalen Community, den Code zu verstehen und sich ggf. in die Entwicklung einzuschalten.

@ISAAKiel ISAAKiel deleted a comment from nmueller18 May 6, 2019
@yesdavid
Copy link
Member Author

yesdavid commented May 6, 2019

Dear @nevrome,

thank you for your response.

First of all I was thinking about a review of the workflow and the implementation of Zimmermann's ideas in general; meaning: is the code representing his method correctly?

Improving the coding style, the algorithms and all of your other suggestions mentioned above should come afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants