Page MenuHomeSoftware Heritage
Differential D5963

origin_search: Add keyword search for instrinsic_metadata keywords/description
ClosedPublic
Actions

Authored by KShivendu on Jul 2 2021, 1:24 PM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Commits
rDSEAf378a989e972: origin_search: Add keyword search for instrinsic_metadata keywords/description
Summary

instrinsic_metadata contains keywords and description which are very useful for
finding desirable origins based on a list of keywords provided by the user.

Diff Detail

Repository
rDSEA Archive search
Branch
metadata-description
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22413
Build 34916: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 34915: arc lint + arc unit

Unit TestsFailed
View All

TimeTest
3 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.search.tests.test_init::test_types[memory-InMemorySearch-None]
mocker = <pytest_mock.plugin.MockerFixture object at 0x7f07778b6940> class_ = 'memory' expected_class = <class 'swh.search.in_memory.InMemorySearch'>, kwargs = None
4 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.search.in_memory::swh.search.in_memory._nested_get
16,236 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.search.tests.test_api_client.TestRemoteSearch::test_journal_client_origin_visit_status_permutation
377 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.search.tests.test_api_client.TestRemoteSearch::test_origin_instrinsic_metadata_license_search
345 msJenkins > .tox.py3.lib.python3.7.site-packages.swh.search.tests.test_api_client.TestRemoteSearch::test_origin_instrinsic_metadata_multiple_field_search
View Full Test Results (1 Failed · 133 Passed · 1 Skipped)

Event Timeline

KShivendu created this revision.Jul 2 2021, 1:24 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptJul 2 2021, 1:24 PM
swh-public-ci added a comment.Jul 2 2021, 1:29 PM

Build has FAILED

Patch application report for D5963 (id=21434)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit c100cd9f5d6dfd7042c0a717cff3fa609e1b3ac1
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description

Link to build: https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/183/
See console output for more information: https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/183/console

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 2 2021, 1:29 PM
Harbormaster failed remote builds in B22412: Diff 21434!
KShivendu updated this revision to Diff 21435.Jul 2 2021, 2:43 PM
  • test_search: Fix failing tests
swh-public-ci added a comment.Jul 2 2021, 2:48 PM

Build has FAILED

Patch application report for D5963 (id=21435)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit f8ded206f956b441750cc17a111fa1c810660dcf
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 12:15:55 2021 +0000

    test_search: Fix failing tests

commit c100cd9f5d6dfd7042c0a717cff3fa609e1b3ac1
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description

Link to build: https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/184/
See console output for more information: https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/184/console

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 2 2021, 2:48 PM
Harbormaster failed remote builds in B22413: Diff 21435!
KShivendu updated this revision to Diff 21442.Jul 2 2021, 4:02 PM
  • origin_search: Fix signature
swh-public-ci added a comment.Jul 2 2021, 4:06 PM

Build is green

Patch application report for D5963 (id=21442)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit 8238dd0163136d45a51fbb44b5c32f0f73eecc8e
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 14:01:43 2021 +0000

    origin_search: Fix signature

commit f8ded206f956b441750cc17a111fa1c810660dcf
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 12:15:55 2021 +0000

    test_search: Fix failing tests

commit c100cd9f5d6dfd7042c0a717cff3fa609e1b3ac1
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description

See https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/185/ for more details.

Harbormaster completed remote builds in B22420: Diff 21442.Jul 2 2021, 4:06 PM
KShivendu requested review of this revision.Jul 2 2021, 4:06 PM
vlorentz added a subscriber: vlorentz.Jul 2 2021, 5:11 PM
vlorentz added inline comments.
swh/search/elasticsearch.py
447

what does ^2 do?

462–471

why these two changes?

swh/search/in_memory.py
447–452

you can sort in-place, no need to copy

KShivendu updated this revision to Diff 21471.Jul 2 2021, 6:17 PM
  • origin_search: Polish the code with get_expansion and other methods
swh-public-ci added a comment.Jul 2 2021, 6:21 PM

Build is green

Patch application report for D5963 (id=21471)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit cd468bdb2fcfc577edf83432f329faa943d46918
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 16:14:08 2021 +0000

    origin_search: Polish the code with get_expansion and other methods

commit 8238dd0163136d45a51fbb44b5c32f0f73eecc8e
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 14:01:43 2021 +0000

    origin_search: Fix signature

commit f8ded206f956b441750cc17a111fa1c810660dcf
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 12:15:55 2021 +0000

    test_search: Fix failing tests

commit c100cd9f5d6dfd7042c0a717cff3fa609e1b3ac1
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description

See https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/187/ for more details.

Harbormaster completed remote builds in B22448: Diff 21471.Jul 2 2021, 6:21 PM
KShivendu marked 3 inline comments as done.Jul 2 2021, 6:21 PM
KShivendu added inline comments.
swh/search/elasticsearch.py
447

Boosts a result score by 2 times (might not be exactly 2x, but that's the idea) if queried keywords are found in that field

swh/search/tests/test_search.py
458–467

These comments are outdated. I'll update them.
But you may check if these test cases are sufficient.

vlorentz accepted this revision.Jul 5 2021, 11:54 AM
This revision is now accepted and ready to land.Jul 5 2021, 11:54 AM
KShivendu updated this revision to Diff 21477.Jul 5 2021, 12:25 PM
KShivendu marked an inline comment as done.
  • Squash
  • Minor polishes
swh-public-ci added a comment.Jul 5 2021, 12:29 PM

Build is green

Patch application report for D5963 (id=21477)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit d2c137d31d482439f17420cfa12c2bba05c6a6ec
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description

See https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/188/ for more details.

Harbormaster completed remote builds in B22456: Diff 21477.Jul 5 2021, 12:29 PM
KShivendu updated this revision to Diff 21478.Jul 5 2021, 12:32 PM

Add commit body

KShivendu edited the summary of this revision. (Show Details)Jul 5 2021, 12:32 PM
swh-public-ci added a comment.Jul 5 2021, 12:36 PM

Build is green

Patch application report for D5963 (id=21478)

Rebasing onto 2e1fb86387...

Current branch diff-target is up to date.
Changes applied before test
commit f378a989e972f1896f4b99fac7e916e4fb133a00
Author: KShivendu <shivendu@iitbhilai.ac.in>
Date:   Fri Jul 2 11:19:08 2021 +0000

    origin_search: Add keyword search for instrinsic_metadata keywords/description
    
    instrinsic_metadata contains keywords and description which are very useful for
    finding desirable origins based on a list of keywords provided by the user.

See https://jenkins.softwareheritage.org/job/DSEA/job/tests-on-diff/189/ for more details.

Harbormaster completed remote builds in B22457: Diff 21478.Jul 5 2021, 12:36 PM
Closed by commit rDSEAf378a989e972: origin_search: Add keyword search for instrinsic_metadata keywords/description (authored by KShivendu). · Explain WhyJul 5 2021, 12:41 PM
This revision was automatically updated to reflect the committed changes.
KShivendu added a commit: rDSEAf378a989e972: origin_search: Add keyword search for instrinsic_metadata keywords/description.

Revision Contents
Changeset List

PathSize
swh/
search/
27 lines
46 lines
3 lines
tests/
58 lines

Diff 21435

View Options

swh/search/elasticsearch.py

Loading...
View Options

swh/search/in_memory.py

Loading...
View Options

swh/search/interface.py

Loading...
View Options

swh/search/tests/test_search.py

Loading...
About · Developer information · Legal