Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Adaptations for Hemnet descriptions / needs renovation#8

Open
sg10 wants to merge 1 commit intoIBM:mainfrom
MovestaDev:hemnet-descriptions-needs-renovation
Open

Adaptations for Hemnet descriptions / needs renovation#8
sg10 wants to merge 1 commit intoIBM:mainfrom
MovestaDev:hemnet-descriptions-needs-renovation

Conversation

@sg10
Copy link

@sg10 sg10 commented Mar 9, 2022

  • add dataset preprocessing
  • human oracle (keyboard input labelling)
  • Swedish BERT
  • adjust constants to avoid BERT OOM
  • fix dependency versions

- add dataset preprocessing
- human oracle (keyboard input labelling)
- Swedish BERT
- adjust constants to avoid BERT OOM
- fix dependency versions
Copy link
Member

@alonh alonh left a comment

Choose a reason for hiding this comment

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

Thanks for using the open source. In general, we want to preserve the existing functionality that reflects the results from the paper. This requires changes to be made a bit differently, namely by adding features and modules rather than changing default values or behaviors. For example, if you want to use PyTorch instead of Tensorflow, you can follow the instructions here.
If you want to enable user input, which is a great idea, please implement it as a different data_access/oracle_data_access so the main experiment code will still be valid.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants