8000 Remove large openie model file by gabrielStanovsky · Pull Request #1756 · allenai/allennlp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Remove large openie model file #1756

Merged
merged 20 commits into from
Sep 12, 2018
Merged

Remove large openie model file #1756

merged 20 commits into from
Sep 12, 2018

Conversation

gabrielStanovsky
Copy link
Contributor
  • Remove large Open IE model file (~30mb), which was pushed accidentally.
  • Use SRL's fixture model instead, in testing the Open IE predictor.

Gabi Stanovsky and others added 20 commits September 6, 2018 18:26
adding Open IE
40e1598
adding Open IE
31abdd8
adding Open IE
f98448b
fixing typo
9f1e1be
Using SRL model instead of a duplicated OIE model, adding an `ignore_…
ba66d12 
…span_metric` parameter to the SRL model to accomodate for Open IE
adding conversion script from open ie extractions to conll format
1cc5d90
minor fixes
6181eb2
style fixes
4022d2a
fixing naming convention
c55587d
fixing naming convention
4b9144f
returning empty dictionary when in get_metrics when ignoring span loss
dba7ab2
minor
45a7905
minor
bb34e6a
adding Open IE predictor to docs
bfbea10
fixing comments for docs
7a0a60d
@gabrielStanovsky
Merge branch 'master' into master
327fc87
Merge branch 'master' of https://github.com/allenai/allennlp
4aaf692
Merge branch 'master' of https://github.com/allenai/allennlp
a74c58b
removing large file
f61fa58
using srl test model instead of the large Open IE file
e3158d7
@gabrielStanovsky
Copy link
Contributor Author

Addresses #1754

matt-gardner
matt-gardner approved these changes Sep 12, 2018
View reviewed changes
Copy link
Contributor
@matt-gardner matt-gardner left a comment

Choose a reason for hiding this comment

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

Thanks!

@matt-gardner matt-gardner merged commit 49f43ec into allenai:master Sep 12, 2018
@huntzhan
Copy link
Contributor

Hi team, this file still exists in the git history. To remove it entirely, the admin could follow this and this guides. The change requires push --force and should be done by admin of repo only.

➜  allennlp tempFile=$(mktemp)
IFS=$'\n'
for commitSHA1 in $(git rev-list --all); do
        git ls-tree -r --long "$commitSHA1" >>"$tempFile"
done
➜  allennlp sort --key 3 "$tempFile" | \
        uniq | \
        sort --key 4 --numeric-sort --reverse

100644 blob 194477803731209e32b5a16191d664e22ce7abad 34068339	allennlp/tests/fixtures/open_information_extraction/serialization/model.tar.gz
100644 blob ea037c6f6e2ab04231056ef148e033677a7e6e4c 8720384	allennlp/tests/fixtures/data/atis/atis.db
100644 blob ad35b7aaaa57cf05bba25a49136061422d659825 1373369	allennlp/service/static/lib/babel.min.js
100644 blob 8e0d573308083a7ee686b397e5bd959ec407d7cb  653066	allennlp/service/static/lib/react-dom.js
100644 blob bb4fac0613755cbada5c903a9e13df57dfa9b4c4  450896	tests/fixtures/elmo/lm_weights.hdf5
100644 blob bb4fac0613755cbada5c903a9e13df57dfa9b4c4  450896	allennlp/tests/fixtures/elmo/lm_weights.hdf5
100644 blob 6ba3daa16db3aa4b9bb26ca34835b931b845db60  450896	tests/fixtures/elmo/lm_weights.hdf5
100644 blob fff1752d3097ddc9ab2e0ff9a8bbdf13b0550be3  435427	tutorials/how_to/visualization_images/bidaf_attention_demo.png
100644 blob 18b950bed672780fdce06d7d6249d14147a0e67c  395659	tutorials/how_to/visualization_images/wikitables_overview.png
100644 blob ed8e8bb91956965ebfba0462f8e9e0483ebff459  207828	tests/fixtures/data/triviaqa-sample.tgz
100644 blob ed8e8bb91956965ebfba0462f8e9e0483ebff459  207828	allennlp/tests/fixtures/data/triviaqa-sample.tgz
100644 blob 987e696a7ead69b16071b8d9d56df57f97120f4b  175690	tutorials/how_to/debugging_images/inspect_variable.png
100644 blob fe862c48571cb01b6a554fdcb1ac400aed9fdd21  160897	tutorials/getting_started/data/cr.train

@nelson-liu
Copy link
Contributor

@huntzhan thanks for bringing this up --- the decision was made to not change the git history through force-pushing (also see #1757 (comment))

@huntzhan
Copy link
Contributor

@nelson-liu got it, thank you for your clarification.

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

@matt-gardner matt-gardner matt-gardner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gabrielStanovsky @huntzhan @nelson-liu @matt-gardner
0