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

Pyserini Onboarding Path: Add an entry in Reproduction Log #1608

Closed
wants to merge 1 commit into from

Conversation

yilinjz
Copy link
Contributor

@yilinjz yilinjz commented Aug 30, 2023

OS: ubuntu 18.04 (WSL)

Hardware:

  • Processor: Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz 2.90 GHz
  • RAM: 32.0 GB
  • System type: 64-bit operating system, x64-based processor

Conda: 23.7.2

Python: (Conda Environment): 3.8.17
Java: (Conda Environment) 11.0.13
Maven: (Conda Environment) 3.9.4

Result: I was able to reproduce the Bi-Encoders experiments (calculate BM25 vector representation for document, vector for query representation as well as find the dot product).

Additional Comment: Switched to Ubuntu as the issue in #1599 persists.

@lintool
Copy link
Member

lintool commented Sep 1, 2023

@yilinjz please resolve conflicts?

@lintool
Copy link
Member

lintool commented Sep 1, 2023

@yilinjz Also, the onboarding path continues...

Okay, that's it for this lesson. Next, you're going to play with an actual dense retrieval model. ...

Can you please work until completion and add repro entries to this PR?

@yilinjz
Copy link
Contributor Author

yilinjz commented Sep 1, 2023

I just realized that I added the rest repro entries of the onboarding path to another branch...
I'll close this PR and make another one with all the repro logs.

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.

2 participants