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

Tasks 922-926:Event2Mind and Coached_Conv_Pref dataset #344

Merged
merged 10 commits into from
Oct 30, 2021

Conversation

yashbhokare
Copy link
Contributor

No description provided.

@danyaljj
Copy link
Contributor

Thanks @yashbhokare! Looks like the tests are failing.
Wondering if you can check the errors?

@aarunku5
Copy link
Contributor

aarunku5 commented Oct 8, 2021

@yashbhokare can you address the test errors? Thanks!

Thanks @yashbhokare! Looks like the tests are failing.
Wondering if you can check the errors?

@danyaljj
Copy link
Contributor

Please make sure to set the "Categories": [""] and "Domains": [""] fields according to the taxonomy presented here.

@yashbhokare
Copy link
Contributor Author

Sure I'll check and update it by EOD.

@danyaljj
Copy link
Contributor

Looks like the tests are failing. I am actually not sure why, but it might be easier to test them locally on your machine:

> python src/test_all.py

@aarunku5
Copy link
Contributor

Looks like the tests are failing. I am actually not sure why, but it might be easier to test them locally on your machine:

> python src/test_all.py

@yashbhokare can you please address this?

@danyaljj danyaljj merged commit f1bad04 into allenai:master Oct 30, 2021
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