Hzfinfdu / FDU-PRML-2023Fall

15 stars 0 forks source link

关于Assignment1_2的几个问题 #5

Open Homer-Z opened 1 year ago

Homer-Z commented 1 year ago

1.在test_linear_reg.py中,我感觉assert_array_almost_equal(reg.coef_, [0])应当改为assert_array_almost_equal(reg.coef_, [[0]])

2.test_knn.py好像有些问题,我试着引入sklearn中的KNeighborsClassifier进行测试,结果也会报错。

3.决策树的criterion选择gini系数时,测试总是不通过,不知道是不是我的问题,如果有同学测试通过了的话能否回复一下?

YifanRuan commented 1 year ago

我也觉得 knn test 有问题. gini 的测试是能通过的

Hzfinfdu commented 1 year ago

KNN的Test不能保证全部通过,可以把自己的观察和不能全部通过的理由写在报告里,不完全通过测试不会影响成绩

YifanRuan commented 1 year ago

谢谢助教. 同时楼主说的第一个问题我认为也存在, 应该把 test_linear_reg.py 第 28 行的 [0] 改成 [[0]], 之前只修改了第 18 行

Hzfinfdu commented 1 year ago

是的,这里是我疏忽了,手动修改下即可