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

doc select drilldown: translate key_vector_expansion #1486

Merged
merged 7 commits into from
Jan 12, 2023

Conversation

HashidaTKS
Copy link
Contributor

Translate key_vector_expansion

@HashidaTKS HashidaTKS changed the title doc select: translate key_vector_expansion doc select drilldown: translate key_vector_expansion Jan 11, 2023
@HashidaTKS
Copy link
Contributor Author

#1476

ベクターをべき集合(power set)に展開して集計(aggregate)します。
このとき、対象のベクターを多重集合(multiset)とみなします。
多重集合とみなすので、同じ値の要素が複数ある場合、それぞれ別の要素とみなします。
This aggregates total with expanding vector to power set.
Copy link
Contributor Author

@HashidaTKS HashidaTKS Jan 11, 2023

Choose a reason for hiding this comment

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

このあたりのvectorやpower setが複数形でもなく冠詞もないのは、これらの単語が概念的に使われているので単数形かつ冠詞なしで大丈夫、ということであっていますか??(意図的に省略されている/なくても正しい、ということであっていますか?)

Copy link
Contributor

Choose a reason for hiding this comment

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

はい
概念的な単語と認識したからですが明確には powersetsかもしれないですね。でもべき集合としては一つなら theか・・・ itsか・・・ですかね

Copy link
Contributor Author

@HashidaTKS HashidaTKS Jan 12, 2023

Choose a reason for hiding this comment

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

なるほど!
具体例では冠詞を付けるなり複数形にするなりして(具体的な例を考えているので)、それ以外の部分では冠詞なしの単数形のままで良さそうかなぁと思いました。
(現状でもそうなっているので、現状のままで大丈夫なのではないかと思いました。)

Copy link
Contributor

Choose a reason for hiding this comment

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

expanding vector to the power set.
が良い気がしてきました・・

Copy link
Contributor Author

Choose a reason for hiding this comment

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

他の部分もvectorは複数形にする、setなどにはtheをつけたほうが良さそうでしょうか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

見返したらvectorで複数形にする必要がありそうなのはここだけでした。。。

Copy link
Contributor

Choose a reason for hiding this comment

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

そうなんです Vectorはここだけぬけてました

power-setはtheつけたほうがいいかもです。
vectorに対応するべき集合だと思うのでいつでも基本は theなんだろうなと思います

Copy link
Contributor Author

Choose a reason for hiding this comment

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

なるほど!

@HashidaTKS
Copy link
Contributor Author

eachの後は単数形が正しそうなので、 each subsetsのようなものをeach subsetに変更。

@HashidaTKS
Copy link
Contributor Author

@yoshimotoyuk

すみません、 #1486 (comment)#1486 (comment) が正しいかどうか確認頂けないでしょうか?

@yoshimotoyuk
Copy link
Contributor

eachの後は単数形が正しそうなので、 each subsetsのようなものをeach subsetに変更。

おっしゃる通りです。
失礼しました!

HashidaTKS and others added 2 commits January 12, 2023 14:35
Co-authored-by: Yukiko Yoshimoto <86996282+yoshimotoyuk@users.noreply.github.com>
@HashidaTKS
Copy link
Contributor Author

@yoshimotoyuk

ありがとうございます。
指摘事項に対応しましたので、これでマージしようと思います。

@HashidaTKS HashidaTKS merged commit 94560b3 into master Jan 12, 2023
@HashidaTKS HashidaTKS deleted the doc-translate-key-vector-expansion branch January 12, 2023 05:54
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.

None yet

2 participants