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

TUKR実装-福永 #15

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

TUKR実装-福永 #15

wants to merge 26 commits into from

Conversation

yuki-fukunaga
Copy link
Contributor

@yuki-fukunaga yuki-fukunaga commented May 17, 2022

  • 関係データについて理解し、関係データとは何か説明できる。
  • TUKRのタスク設定について理解し、数式やアルゴリズムの各ステップを説明でき
    る。
  • data_scrach.pyの穴埋めを行い関係データの人工データを作成する。
  • UKRの実装ファイルを改良してTUKRを実装する。
  • 正則化項の実装と確認。
  • 欠損データを実装する
  • 手動微分で実装する。
  • 描画方法を変更する(準備中)
  • クロステストの実装(準備中)
  • 実データでの実装。(動物データやNIPSデータなど)
  • 手動微分を数式的に導くことができる。

@TetraMiyazaki
Copy link
Contributor

TetraMiyazaki commented May 17, 2022

ミニマムサクセスだけじゃなくてスタンダードもやってくださいね.ミニマムは研究を進めていくための必要条件というだけで,自分の研究を対外的に説明するとか読み物としてまとめる時は,深く広い理解が必要です.(UKRのスタンダードが終わってないならUKRのスタンダードからやるのは一つありです)

Copy link
Contributor

@nahoando nahoando left a comment

Choose a reason for hiding this comment

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

wiki,readmeを参考に自分の中でのミニマム、スタンダード、エクストラの設定をしておきましょう。UKRで確認し損ねた内容があるならそちらも一緒に確認してみるといいですね。作業にひと段落ついたらプッシュをするのを忘れずに!

@yuki-fukunaga
Copy link
Contributor Author

スクリーンショット 2022-05-19 15 08 51

@yuki-fukunaga
Copy link
Contributor Author

TUKR
一応動いたよー

@yuki-fukunaga
Copy link
Contributor Author

TUKR
一様分布に修正したやつー

@TetraMiyazaki
Copy link
Contributor

TetraMiyazaki commented May 27, 2022

image
ここなんで普通に単調減少しないでボコってなってるやよ〜どう思いますかやよ〜

@yuki-fukunaga
Copy link
Contributor Author

UKR.1.mp4

@TetraMiyazaki
Copy link
Contributor

観測空間のほうが描画されてないですが,写像がちゃんと$f(u_i, v_j) = x_{ij}$のようにデータ点に近くになっていることはどうやって確認していますか?

@yuki-fukunaga
Copy link
Contributor Author

TUKR.mp4

Copy link
Contributor

@TetraMiyazaki TetraMiyazaki left a comment

Choose a reason for hiding this comment

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

ここで,ユーザーの潜在空間と商品の潜在空間は別物だからcolormap1, 2と用意するのはいいことですが,結局いれているのは青 -> 緑の値のrgb をいれていることになるので,もったいないです.それぞれちゃんと rgb用意してあげるといいですよ.

    colormap = X[:, :, 0]
    colormap1 = U[:, 0]
    colormap2 = V[:, 0]

Copy link
Contributor

@TetraMiyazaki TetraMiyazaki left a comment

Choose a reason for hiding this comment

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

#15 (review)
ここが解決したらapproveします

@yuki-fukunaga
Copy link
Contributor Author

TUKR2.mp4

TetraMiyazaki
TetraMiyazaki previously approved these changes Jun 7, 2022
Copy link
Contributor

@TetraMiyazaki TetraMiyazaki left a comment

Choose a reason for hiding this comment

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

色付けできたみたいなのでApproveします

@TetraMiyazaki
Copy link
Contributor

ローカルでcoflictを頑張って解決する必要がありそうですね

@hirowatari-s hirowatari-s removed their request for review June 12, 2023 06:20
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