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

Erstellen von Histogrammen auf Basis von Statistiken #249

Merged
merged 28 commits into from
Sep 29, 2020

Conversation

LukasEberle
Copy link

Ich starte schonmal einen pull request. Die Methode build ist noch nicht implementiert, aber ich hätte gerne schon etwas Feedback. Außerdem können wir dann morgen etwas fundierter darüber reden.

Copy link
Owner

@mrapp-ke mrapp-ke left a comment

Choose a reason for hiding this comment

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

Ich starte schonmal einen pull request. Die Methode build ist noch nicht implementiert, aber ich hätte gerne schon etwas Feedback.

Ich habe Kommentare zu einigen Codestellen hinzugefügt.

Außerdem können wir dann morgen etwas fundierter darüber reden.

Wir haben den nächsten Termin offen gelassen, oder? Wenn du aber morgen darüber sprechen willst können wir das um 14 Uhr machen.

python/boomer/common/cpp/binning.h Outdated Show resolved Hide resolved
python/boomer/common/cpp/binning.h Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/label_wise_statistics.h Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/example_wise_statistics.h Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/label_wise_statistics.h Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/label_wise_statistics.cpp Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/label_wise_statistics.cpp Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/label_wise_statistics.cpp Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/example_wise_statistics.cpp Outdated Show resolved Hide resolved
python/boomer/boosting/cpp/example_wise_statistics.cpp Outdated Show resolved Hide resolved
@mrapp-ke mrapp-ke changed the title Histogram creation Erstellen von Histogrammen auf Basis von Statistiken Sep 28, 2020
@LukasEberle
Copy link
Author

Wir haben den nächsten Termin offen gelassen, oder? Wenn du aber morgen darüber sprechen willst können wir das um 14 Uhr machen.

Wäre gut wenn wir da morgen um 14 Uhr drüber reden können. Die Kommentare haben mir geholfen ein klareres Bild zu bekommen, ich schaue das ich damit bis morgen möglichst weit komme.

@mrapp-ke
Copy link
Owner

Wir haben den nächsten Termin offen gelassen, oder? Wenn du aber morgen darüber sprechen willst können wir das um 14 Uhr machen.

Wäre gut wenn wir da morgen um 14 Uhr drüber reden können. Die Kommentare haben mir geholfen ein klareres Bild zu bekommen, ich schaue das ich damit bis morgen möglichst weit komme.

Okay, dann machen wir das so.

@LukasEberle
Copy link
Author

Ich habe mal die Kommentare abgearbeitet und würde dann später ins besondere über den Destruktor reden, weil ich mir da nicht sicher bin ob ich das richtig gemacht habe, und über die beiden onBinUpdate Methoden, bei denen ich mir relativ sicher bin, aber es ist der größte Implementierungsschritt.

@mrapp-ke
Copy link
Owner

Ich habe noch Kommentare hinzugefügt und den Code leicht umformatiert so dass es dem Rest entspricht. Die Implementierung für den SeCo-Algorithmus habe ich jetzt doch erst einmal weggelassen, da in diesem Fall die Dinge etwas komplizierter sind.

@mrapp-ke mrapp-ke merged commit 02555a8 into approximate-conditions Sep 29, 2020
@mrapp-ke mrapp-ke deleted the histogram-creation branch September 29, 2020 14:27
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.

Datenstruktur für Repräsentation eines Bins Erstellen von Histogrammen auf Basis von Statistiken
3 participants