search

ponder-lab / Imperative-DL-Study-Web-App

0 stars 0 forks source link

delete categorizations in test setup #156

Closed ZhongweiL closed 2 years ago

ZhongweiL commented 2 years ago

resolves the bug in PR #148 with deleting categorizers by making sure the categorization table is empty in test setup

khatchad commented 2 years ago

Thanks.

khatchad commented 2 years ago

@ZhongweiL , please watch the tests and let me know if they pass.

ZhongweiL commented 2 years ago

@khatchad They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django.

khatchad commented 2 years ago

Failing: https://github.com/ponder-lab/Imperative-DL-Study-Web-App/runs/4450225118?check_suite_focus=true

khatchad commented 2 years ago

What changes haven't you implemented?

Raffi Khatchadourian @.***> Assistant Professor, Computer Science, Hunter College Doctoral Faculty of the Graduate School and University Center's Ph.D. Program in Computer Science City University of New York (CUNY) http://cs.hunter.cuny.edu/~Raffi.Khatchadourian99

On Dec 7, 2021 5:51 PM, Zhongwei Li @.***> wrote:

@khatchadhttps://github.com/khatchad They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ponder-lab/Imperative-DL-Study-Web-App/pull/156#issuecomment-988318688, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPUGP37GTFSUFUNSCKLVATUP2FXXANCNFSM5JSLMNHA.

khatchad commented 2 years ago

Please investigate.

Raffi Khatchadourian @.***> Assistant Professor, Computer Science, Hunter College Doctoral Faculty of the Graduate School and University Center's Ph.D. Program in Computer Science City University of New York (CUNY) http://cs.hunter.cuny.edu/~Raffi.Khatchadourian99

On Dec 7, 2021 5:51 PM, Zhongwei Li @.***> wrote:

@khatchadhttps://github.com/khatchad They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ponder-lab/Imperative-DL-Study-Web-App/pull/156#issuecomment-988318688, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPUGP37GTFSUFUNSCKLVATUP2FXXANCNFSM5JSLMNHA.

ZhongweiL commented 2 years ago

What changes haven't you implemented? Raffi Khatchadourian @.> Assistant Professor, Computer Science, Hunter College Doctoral Faculty of the Graduate School and University Center's Ph.D. Program in Computer Science City University of New York (CUNY) http://cs.hunter.cuny.edu/~Raffi.Khatchadourian99 On Dec 7, 2021 5:51 PM, Zhongwei Li @.> wrote: @khatchadhttps://github.com/khatchad They shouldn't pass because we're writing tests before implementing the changes, but I don't think I understand the error message, they seem like some low-level issues with Django. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#156 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPUGP37GTFSUFUNSCKLVATUP2FXXANCNFSM5JSLMNHA.

@khatchad Never mind, the changes for this test have been implemented in the model and merged, but I don't think it has been migrated to the remote database, which may indicate that the automatic deployment has failed. Or it might be because that the automatic deployment was added after the change was merged.