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

Add to onboarding reproduction logs (#2614) #2614

Merged
merged 4 commits into from
Sep 27, 2024

Conversation

Hossein-Molaeian
Copy link
Contributor

windows OS with conda environment. a bit of adjustment needed for indexing and retrieval (xms to 512mb and xmx to 4gb eventhough my machine runs on 24gb of ram, also 4 threads in the command instead of 9). I encountered more errors on Windows than on Linux, but most of them can be easily fixed with a quick online search. Other than these, everything worked just fine.

@lintool
Copy link
Member

lintool commented Sep 26, 2024

hi @Hossein-Molaeian thanks for your PR. Given that we have relatively few Windows users, please remove your Windows commands on this PR - we'll leave it as a exercise to the user to figure out the equivalent commands. Thanks!

@Hossein-Molaeian Hossein-Molaeian changed the title Add to onboarding reproduction logs + equivalent Windows commands (#2614) Add to onboarding reproduction logs (#2614) Sep 27, 2024
@lintool lintool merged commit 25523b4 into castorini:master Sep 27, 2024
1 check passed
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