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

Unit tests for DA #20

Open
oyamad opened this issue Jun 13, 2016 · 9 comments
Open

Unit tests for DA #20

oyamad opened this issue Jun 13, 2016 · 9 comments

Comments

@oyamad
Copy link
Member

oyamad commented Jun 13, 2016

テストを作ったので,それが通るようにコードを書いてください.
単体テスト

わからないことがあったらここに書いてください.

BaseTestNext がないといわれたら

Pkg.add("BaseTestNext")

でインストールする.

@oyamad
Copy link
Member Author

oyamad commented Jun 23, 2016

Many-to-one 用にもテストを作ったので,それが通るようにコードを書いてください.
単体テスト

わからないことがあったらここに書いてください.

@13tsuyoshi
Copy link
Contributor

僕のmatchingのレポジトリーのMany_to_one.ipynbのファイルであるように、Many-to-one 用のテストファイルが開かないトラブルに陥っています。

@oyamad
Copy link
Member Author

oyamad commented Jun 24, 2016

test_deferred_acceptance.jl がちゃんとダウンロードできていないのではないでしょうか.(前回の同名ファイルに上書きしてOK.)

@Masaokb
Copy link
Contributor

Masaokb commented Jun 24, 2016

先生のone to oneのソースコードってどっかで見られましたっけ

@13tsuyoshi
Copy link
Contributor

include("test_Many_to_one.jl ")と、test_Many_to_one.jlの後に一文字分余計なインデントが入っていたのが原因でした。お騒がせしました。

@oyamad
Copy link
Member Author

oyamad commented Jun 24, 2016

先生のone to oneのソースコードってどっかで見られましたっけ

ここにあります.

@oyamad
Copy link
Member Author

oyamad commented Jun 25, 2016

test_deferred_acceptance.jl を修正したので,すでにダウンロードした人はダウンロードし直してください.

(各大学にマッチした学生の番号は若い順番に並んでいる必要はないが,そのことに対応してなかったので修正した.)

@Masaokb
Copy link
Contributor

Masaokb commented Jul 2, 2016

many to manyのテストって用意されていましたっけ

@oyamad
Copy link
Member Author

oyamad commented Jul 2, 2016

many to manyのテストって用意されていましたっけ

まだ作ってないです.何なら作っていただいても.

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

No branches or pull requests

3 participants