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

implemented var and std #205

Merged
merged 5 commits into from
Dec 8, 2023
Merged

implemented var and std #205

merged 5 commits into from
Dec 8, 2023

Conversation

vserret
Copy link
Contributor

@vserret vserret commented Dec 1, 2023

No description provided.

@vserret vserret marked this pull request as draft December 1, 2023 10:02
victoria de sainte agathe added 2 commits December 4, 2023 09:23
@vserret vserret marked this pull request as ready for review December 4, 2023 08:37
@vserret vserret requested a review from ngrislain December 4, 2023 08:37
@@ -150,6 +150,10 @@ impl PUPRelation {
let one_col = format!("_ONE_{}", col_name);
let sum_col = format!("_SUM_{}", col_name);
let count_col = format!("_COUNT_{}", col_name);
let square_col = format!("_SQUARE_{}", col_name);
let one_square_col = format!("_ONE_{}", square_col);
Copy link
Contributor

Choose a reason for hiding this comment

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

Pourquoi a-t-on besoin d'un one différent ?

Copy link
Contributor

Choose a reason for hiding this comment

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

idem pour count ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

parce que l'on veut deux counts bruités différents dans l'estimateur

@vserret vserret marked this pull request as draft December 7, 2023 11:19
@vserret vserret marked this pull request as ready for review December 7, 2023 11:47
let square_col = format!("_SQUARE_{}", col_name);
let one_square_col = format!("_ONE_{}", square_col);
let sum_square_col = format!("_SUM_{}", square_col);
let count_square_col = format!("_COUNT_{}", square_col);
Copy link
Contributor

Choose a reason for hiding this comment

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

Dis moi quand tu as pu supprimer cela

@ngrislain ngrislain merged commit bf3ea48 into main Dec 8, 2023
1 check passed
@ngrislain ngrislain deleted the impl_more_agg_functions branch December 8, 2023 14:18
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.

2 participants